[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