[PATCH] D149587: InstSimplify: Simplifications for ldexp

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 10 05:28:07 PDT 2023


foad added a comment.

I still think removing support for amdgcn_ldexp should be in the patch that removes amdgcn_ldexp.

I still think every simplification guarded by !IsStrict should have a comment saying why, even if it's just "to be conservative because we're not *sure* that it's fpstrict-safe".


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

https://reviews.llvm.org/D149587



More information about the llvm-commits mailing list