[PATCH] D159275: libclc: Fix signed integer underflow in abs_diff

Romaric Jodin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 31 06:09:32 PDT 2023


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

clspv is also implementing `abs_diff` like that, and it is OpenCL compliant:
https://github.com/google/clspv/blob/b14ef325ef31894b0aaeca9a84e65b10ae53a816/lib/ReplaceOpenCLBuiltinPass.cpp#L1148
https://godbolt.org/z/x7vzd85os


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159275



More information about the llvm-commits mailing list