OP_INLINED_CALL are there only as a sort of annotation for debugging purpose. Their presence should thus not block the packing of basic blocks. Fix this by ignoring OP_INLINED_CALL when trying to pack a basic block. Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> --- flow.c | 1 + validation/optim/call-inlined.c | 30 ++++++++++++++++++++++++++++++ 2 files changed, 31 insertions(+) create mode 100644 validation/optim/call-inlined.c diff --git a/flow.c b/flow.c index 237c9f1fa..ec7e3f22c 100644 --- a/flow.c +++ b/flow.c @@ -949,6 +949,7 @@ void pack_basic_blocks(struct entrypoint *ep) continue; switch (first->opcode) { case OP_NOP: case OP_LNOP: case OP_SNOP: + case OP_INLINED_CALL: continue; case OP_CBR: case OP_BR: { diff --git a/validation/optim/call-inlined.c b/validation/optim/call-inlined.c new file mode 100644 index 000000000..00698a4b1 --- /dev/null +++ b/validation/optim/call-inlined.c @@ -0,0 +1,30 @@ +static const char messg[] = "def"; + +static inline int add(int a, int b) +{ + return a + b; +} + +int foo(int a, int b, int p) +{ + if (p) { + add(a + b, 1); + return p; + } + return 0; +} + +/* + * check-name: call-inlined + * check-command: test-linearize -Wno-decl $file + * + * check-output-start +foo: +.L0: + <entry-point> + select.32 %r9 <- %arg3, %arg3, $0 + ret.32 %r9 + + + * check-output-end + */ -- 2.12.0 -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html