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

Mikhail Maltsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 15 03:27:13 PST 2018


miyuki 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:
----------------
efriedma wrote:
> Shouldn't this be `TLI.getBooleanContents(VT)`? None of the operands of a SETCC are boolean.
As far as I understand, the parameter of `getBooleanContents` is the type of values being compared rather than the result type. At least, other uses of `getBooleanContents` in https://reviews.llvm.org/rL212697 suggest so.


https://reviews.llvm.org/D43282





More information about the llvm-commits mailing list