[PATCH] D95373: Replace vector intrinsics with call to vector library

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D95373#2525539 <https://reviews.llvm.org/D95373#2525539>, @LukasSommerTu wrote:

> Thanks to everyone for your feedback so far!
>
>> I haven't actually looked at the patch details, but if there's agreement that it's ok to leave it as an IR transform, then it should be a small adjustment to make this an IR codegen pass. It would be similar to the existing ExpandReductions pass.
>
> As this solution (i.e., keeping it as an IR pass, but move it to be an IR codegen pass) seems to be acceptable, I would change the implementation accordingly and update the patch.
>
> @lebedev.ri, @fhahn, @spatel, @venkataramanan.kumar.llvm, @craig.topper: Any objections?

No objections from me.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95373/new/

https://reviews.llvm.org/D95373