[llvm] [InstCombine] Do not require nsz for reassociating fmul (PR #93698)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Thu May 30 01:19:39 PDT 2024


jayfoad wrote:

> > You shouldn't need `nsz` for reassociating `fadd` either. The sign of a 0.0 in default rounding mode (which `fadd` implies) is the bitwise and of the sign bits of the inputs, and consequently, it's associative in that case.
> 
> Yeah, I agree, but I had to think hard about it. The only way (a+b)+c can be -0 is if a b c are all -0, in which case reassociating obviously doesn't change the result.
> 
> Anyway I would do that as a separate PR.

#93782

https://github.com/llvm/llvm-project/pull/93698


More information about the llvm-commits mailing list