[PATCH] D42737: [LegalizeDAG] Truncate condition operand of ISD::SELECT
Elena Demikhovsky via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 5 22:58:52 PST 2018
delena added inline comments.
================
Comment at: lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:368
+ // Truncate the condition if needed
+ auto BoolVT = getSetCCResultType(CondVT);
----------------
Can the condition be wider than getSetCCResultType?
================
Comment at: test/CodeGen/AArch64/expand-select.ll:4
+; Check that we don't crash
+; RUN: llc -mtriple=aarch64-unknown-linux-gnu -O3 %s -o -
+
----------------
I think you should put here regular CHECKs, like in all other llc tests.
https://reviews.llvm.org/D42737
More information about the llvm-commits
mailing list