[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