We agreed in D10555 that, as long as we don't affect -O0 codegen too much, it's OK to use native constructs rather than intrinsics. Let's test that, starting with AVX2 here. I'll add others if people are fine with the idea, or have objections to the mixed testing.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Definitely like this idea - hopefully it'll make it safer to remove unnecessary builtins.
avx2-builtins-codegen.c | ||
---|---|---|
182 ↗ | (On Diff #32732) | Please can you add a comment explaining the FIXME? |
Comment Actions
LGTM with a suitable FIXME comment. Email me if you want to split the work on adding this to the other SSE/AVX builtin tests.
Comment Actions
Thanks Simon! Replaced the FIXMEs with proper tests and committed.
Whenever I have time I'll have a look at the others, filed PR24580.