Adding more SIMD constant folding support #82190
Conversation
dotnet-issue-labeler
bot
added
the
area-CodeGen-coreclr
ghost
assigned 
|
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch, @kunalspathak Issue DetailsThis adds support for other primtive unary and binary operations but does not cover relational operators yet.
|
tannergooding
force-pushed
the
cns_vec-fold
branch
2 times, most recently
from
2a0fcd2 to
29117c7
Compare
tannergooding
force-pushed
the
cns_vec-fold
branch
from
29117c7 to
2171541
Compare
tannergooding
force-pushed
the
cns_vec-fold
branch
from
9af1199 to
e80c008
Compare
|
spmi-replay failures are #82397 |
|
/azp run runtime-coreclr jitstress-isas-x86, runtime-coreclr jitstress-isas-arm, runtime-coreclr outerloop, runtime-coreclr jitstress, runtime-coreclr libraries-jitstress, Fuzzlyn |
|
Azure Pipelines successfully started running 6 pipeline(s). |
|
CC. @dotnet/jit-contrib More SIMD constant folding support, covering the other basic binary operators (but not comparisons). All of these have equivalent scalar handling. |
src/coreclr/jit/simd.h
Outdated
| uint32_t arg0Bits = *reinterpret_cast<uint32_t*>(&arg0); | ||
| uint32_t arg1Bits = *reinterpret_cast<uint32_t*>(&arg1); |
There was a problem hiding this comment.
This is technically UB -- can we use memcpy to be safe?
There was a problem hiding this comment.
Will update the new occurrences in this PR.
Notably we're doing this in many places throughout the JIT, so its a more general thing that should be fixed (separately, if it's a concern).
This adds support for other primitive unary and binary operations but does not cover relational operators yet.
Tests still need to be added as well.