Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
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 ↗ | (On Diff #119074) | 'atleast' should be two words |
830 ↗ | (On Diff #119074) | Line up the last argument with the end of the opening parenthese above |
842 ↗ | (On Diff #119074) | These two lines are indented one space too far. |
875 ↗ | (On Diff #119074) | Move this bitcast into the ApplyX86MaskOn1BitsVec. The same one appears at the second call site. |
1050 ↗ | (On Diff #119074) | Extract space between Pred and comma |
Comment Actions
LGTM
lib/IR/AutoUpgrade.cpp | ||
---|---|---|
847 ↗ | (On Diff #121697) | 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 |