Depends on D52755.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
test/CodeGen/WebAssembly/simd-arith.ll | ||
---|---|---|
2 ↗ | (On Diff #168040) | I assume you removed this because you didn't add fast-isel support? You can still keep it as long as you don' have the -fast-isel-abort flag, because if a block cannot be selected with fast-isel it will fall back to SelectionDAG. |
test/CodeGen/WebAssembly/simd-arith.ll | ||
---|---|---|
2 ↗ | (On Diff #168040) | Good to know! I'm not sure that's useful for a test, though, since we want to be explicitly testing fast-isel or not testing it at all. |
test/CodeGen/WebAssembly/simd-arith.ll | ||
---|---|---|
2 ↗ | (On Diff #168040) | In that case we should add -fast-isel-abort to the llc command lines where we have -fast-isel. |
test/CodeGen/WebAssembly/simd-arith.ll | ||
---|---|---|
2 ↗ | (On Diff #168040) | And, even if you don't have fast-isel implemented for some of these instructions, it can still be useful to have a test that allows fallback. If the codegen is different between fast and DAG ISel, then the test actually will fail if fast-isel bails unexpectedly. And even if not you can still cover the code (which could catch asserts, etc) even if you can't ensure that it runs |
lib/Target/WebAssembly/WebAssemblyISelLowering.cpp | ||
---|---|---|
971 ↗ | (On Diff #168040) | How about adding newlines between categories, like default / bitselect / anytrue+alltrue / lsda ? |
lib/Target/WebAssembly/WebAssemblyInstrSIMD.td | ||
203 ↗ | (On Diff #168040) | Maybe start (vec_t ... from the next line? |
466 ↗ | (On Diff #168040) | Does this handle the case of all different commutative permutations of or and and? like,
It would be also good to add tests for some of other patterns too. |