InstSimplify: fold 0/X to 0 if nnan and nsz
milseman at apple.com
Sun Feb 22 20:49:42 PST 2015
Add a comment before the transformation, otherwise LGTM.
> On Feb 20, 2015, at 7:17 PM, Fiona Glaser <fglaser at apple.com> wrote:
> Per the subject! This came up in real production code, so it’s not merely an artificial test case. I was kind of surprised this combine seemed to be missing; is there some reason it’s not legitimate, or was it just forgotten?
> (Also, fast-math logic: we need nnan because 0 / 0 can cause NaN, and we need nsz because we don’t know the sign of X, so 0/X could result in either 0 or -0 depending on the sign of X.)
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
More information about the llvm-commits