[PATCH] D86260: [CodeGen] Postprocess PHI nodes for callbr
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 20 11:03:48 PDT 2020
craig.topper added inline comments.
================
Comment at: llvm/test/CodeGen/X86/callbr-asm-phi-nodes.ll:9
+; CHECK: # %bb.0:
+; CHECK-NEXT: movl %gs:0, %eax
+; CHECK-NEXT: #APP
----------------
nickdesaulniers wrote:
> What was the input C program, was it precisely https://godbolt.org/z/zqK1K7 from https://bugs.llvm.org/show_bug.cgi?id=47242. If so, this `mov` into `%eax` looks problematic; it gets overwritten on both branches of the INLINEASM_BR, and looks like a dead store to me.
I think its there because the value of @var is an input to the inline asm. But the asm doesn't use it?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86260/new/
https://reviews.llvm.org/D86260
More information about the llvm-commits
mailing list