This reverts commit 940efa4f6981964c95a69ff412378722438607bf.
Reason for revert: it breaks TestOpenCLLocalEnergyMinimizer test of OpenMM.
Paths
| Differential D117797
Revert "[amdgpu] Improve the from f32 to i64." AbandonedPublic Authored by kzhuravl on Jan 20 2022, 8:57 AM.
Details Summary This reverts commit 940efa4f6981964c95a69ff412378722438607bf. Reason for revert: it breaks TestOpenCLLocalEnergyMinimizer test of OpenMM.
Diff Detail Event TimelineHerald added subscribers: foad, kerbowa, hiraditya and 7 others. · View Herald TranscriptJan 20 2022, 8:57 AM Comment Actions
Incorrect results being produced by the kernel, so the comparison with host results fail. Comment Actions Is anyone going to debug this? Or can you at least narrow it down to an input that produces the wrong result? Comment Actions
But on what specific input/output values are broken? This revision now requires changes to proceed.Nov 16 2022, 3:15 PM
Revision Contents
Diff 401658 llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-fptosi.mir
llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-fptoui.mir
llvm/test/CodeGen/AMDGPU/fp_to_sint.ll
llvm/test/CodeGen/AMDGPU/fp_to_uint.ll
|
clang-format: please reformat the code