[PATCH] D36654: [X86] Correct two places that seemed to be mishandling X86ISD::UMUL

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 13 13:22:50 PDT 2017


RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:26920
+    if (Op.getResNo() == 2)
+      goto bool_result;
+    break;
----------------
Probably a lot cleaner just to set Known.Zero.setBitsFrom(1); here instead of using goto


https://reviews.llvm.org/D36654





More information about the llvm-commits mailing list