Depends on D52805.
Details
Diff Detail
- Repository
- rL LLVM
- Build Status
Buildable 23421 Build 23420: arc lint + arc unit
Event Timeline
lib/Target/WebAssembly/WebAssemblyISelLowering.cpp | ||
---|---|---|
993 | Newline here? :3 |
test/CodeGen/WebAssembly/simd-arith.ll | ||
---|---|---|
6 | Do we need both the slow and fast versions for NO-SIMD128 too? Aren't they the same anyway? |
test/CodeGen/WebAssembly/simd-arith.ll | ||
---|---|---|
6 | Oh what I meant is... do we need to test both with and without fastisel for NO-SIMD128? With NO-SIMD128 we are just testing the pre-existing scalar expansion behavior and checking the SIMD instructions do not kick in. |
test/CodeGen/WebAssembly/simd-arith.ll | ||
---|---|---|
6 | Sounds like fast isel is responsible for complying with architecture features itself, so we still need this test. |
Newline here? :3