[llvm] [RISCV] Fix lowering of negative zero with Zdinx 32-bit (PR #71869)

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 14 07:58:40 PST 2023


preames wrote:

Glancing at the code post review, don't we have the same problem a couple of lines above for the FLI/ZFA codepath?  Or is dinx and zfa an illegal combination in some way?

And unrelated, but with dinx and zbs, couldn't we be better just using a bseti to materialize the sign that using the dedicated sign flip instruction?  Not sure it hugely matters, but maybe?  

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


More information about the llvm-commits mailing list