This specialization allows sharing more code where an AXPY follows naturally
in cases where an OUTERPRODUCT on a scalar would be generated.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Thanks for adopting what we discussed previously and special casing the impl!
Overall this looks good, I was expecting to see much more red on the VectorTransforms side as well as folding the AXPY into OuterProduct option.
Is this left for a future revision?
It seems the refactoring would be more compelling if it had all the red it deserves :)
Comment Actions
Yes, the merging of the outerproduct/AXPY lowering is already done, but in a separate CL.
I like to keep CL's small to make the review easier, and extending the semantics of outer with axpy seemed a logically contained unit.
Refactoring to use the new operation seemed another logical unit.