[PATCH] D43282: [LegalizeDAG] Fix legalization of SETCC

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 14 14:15:24 PST 2018


efriedma added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:3769
     int TrueValue;
-    switch (TLI.getBooleanContents(Tmp1->getValueType(0))) {
+    switch (TLI.getBooleanContents(Tmp1.getValueType())) {
     case TargetLowering::ZeroOrOneBooleanContent:
----------------
Shouldn't this be `TLI.getBooleanContents(VT)`? None of the operands of a SETCC are boolean.


https://reviews.llvm.org/D43282





More information about the llvm-commits mailing list