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 | ||
|---|---|---|
| 829 | 'atleast' should be two words | |
| 831 | Line up the last argument with the end of the opening parenthese above | |
| 843 | These two lines are indented one space too far. | |
| 875–876 | 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 | ||
|---|---|---|
| 847 | 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