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

Steve Canon via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 19 07:46:30 PDT 2021


scanon added a comment.

What's the rationale for making abs undefined on the minimum value? AFAIK every actual simd implementation defines the result and they agree (and even if one didn't, it would be pretty easy to get the "right" result. Introducing UB here just seems like punishing users for no reason.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111986



More information about the cfe-commits mailing list