[PATCH] D33168: Fix PR33028

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 17 06:10:00 PDT 2017


RKSimon added a comment.

I think this is sound, but I'm not an expert on any of this. Any other comments?



================
Comment at: lib/Target/X86/X86ISelLowering.cpp:24758
   // sinkMBB:
   // EAX is live into the sinkMBB
+  BuildMI(*sinkMBB, sinkMBB->begin(), DL, TII->get(X86::PHI), DstReg)
----------------
Isn't this comment for fallMBB now?


Repository:
  rL LLVM

https://reviews.llvm.org/D33168





More information about the llvm-commits mailing list