[llvm] [InstCombine] Fold more 'fcmp' 'select' instrs idioms into 'fabs' (PR #83381)
Andy Kaylor via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 21 11:42:48 PDT 2024
andykaylor wrote:
> One annoying bit about handling this as a parameter attribute is we can't attach the same to other arbitrary sources (e.g. load) which should behave exactly the same. We would need another mirror metadata
Yes, not being able set the flag on loads is a problem. In cases where SROA is going to eliminate a store followed immediately by a load, but I don't see any reason the `nsz` property wouldn't also apply to any loads of floating-point data from other sources within the scope where the setting is active.
https://github.com/llvm/llvm-project/pull/83381
More information about the llvm-commits
mailing list