[llvm] AMDGPU: Replace sqrt OpenCL libcalls with llvm.sqrt (PR #74197)
Stanislav Mekhanoshin via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 15 00:31:26 PST 2023
rampitec wrote:
> nsz doesn't do anything for sqrt lowering.
I know it does not. Why not remove it?
I will feel much more confident if this one is fixed from nsz nonsense and extra 'fast' test added, because who knows what fast will mean tomorrow? Anyway, formally to have no regressions fast must resolve to a native op, so this test is formally needed.
https://github.com/llvm/llvm-project/pull/74197
More information about the llvm-commits
mailing list