[PATCH] D45146: [x86] Introduce a pass to begin more systematically fixing PR36028 and similar issues.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 3 23:00:24 PDT 2018


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86FlagsCopyLowering.cpp:615
+    Cond = X86::COND_B; // CF == 1
+    Addend = 127;
+    break;
----------------
I think these Addend values are backwards. 255 would work for carry, and 127 would work for overflow


Repository:
  rL LLVM

https://reviews.llvm.org/D45146





More information about the llvm-commits mailing list