[PATCH] D111986: [Clang] Add elementwise abs builtin.

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:3109
+      Result = Builder.CreateBinaryIntrinsic(
+          llvm::Intrinsic::abs, Op0, Builder.getFalse(), nullptr, "elt.abs");
+    else
----------------
craig.topper wrote:
> Did we discuss that this is different than __builtin_abs which is undefined for INT_MIN?
I don't think we add nsw to vector signed integer subtract, so I guess using false here is consistent.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111986