Details
Diff Detail
Event Timeline
Added fast-isel tests.
Note that for avx512bw there is an open bug that responsible for that huge code size.
lib/IR/AutoUpgrade.cpp | ||
---|---|---|
828 | 'atleast' should be two words | |
830 | Line up the last argument with the end of the opening parenthese above | |
842 | These two lines are indented one space too far. | |
875 | Move this bitcast into the ApplyX86MaskOn1BitsVec. The same one appears at the second call site. | |
1050 | Extract space between Pred and comma |
LGTM
lib/IR/AutoUpgrade.cpp | ||
---|---|---|
846 | You can use Builder.getIntNTy(std::max(NumElts, 8U)) here and you won't need the Context argument. Builder already has a reference to the Context internally and getIntNTy is just a helper to call IntegerType::get using that reference |
'atleast' should be two words