[PATCH] D87101: [X86] Update SSE/AVX ABS intrinsics to emit llvm.abs.* (PR46851)

Sanjay Patel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 4 08:29:48 PDT 2020


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM - I think the odds of code that is using intrinsic vector abs inter-mixing with auto-vectorized code using the abs idiom are small.

I noticed one other LLVM codegen test that might want to be updated to use the IR intrinsics to be more relevant -- llvm-project/llvm/test/CodeGen/X86/combine-abs.ll. That can be done with this patch or later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87101



More information about the cfe-commits mailing list