Essentially the same as the signed variants from D88259. Also includes an attempt to clean up the lowering function.
Details
Details
Diff Detail
Diff Detail
Unit Tests
Unit Tests
Event Timeline
llvm/test/CodeGen/AArch64/sve-fixed-length-int-reduce.ll | ||
---|---|---|
1042 | Just noticed this. Will correct... |
Comment Actions
Just a heads up that @kmclaughlin is starting to look at legalisation/lowering for scalable vector types. From an operation legalisation point of view you've done most of the plumbing so hopefully any toe treading will be minimal.
Comment Actions
Ok, no problem. I still have to finish the fixed FP reductions. If I get in your way, just shout.
Or, if @kmclaughlin would like, I can add the Scalable lowerings while I'm here. Assuming you don't have downstream changes prepared already...
Comment Actions
There's the possibility of some type legalisation changes as well, so best to leave the scalable lowerings for @kmclaughlin.
clang-format: please reformat the code