[llvm] [NVPTX] deprecate nvvm.rotate.* intrinsics, cleanup funnel-shift handling (PR #107655)
Alex MacLean via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 23 12:03:35 PDT 2024
AlexMaclean wrote:
> Are you saying that your patch produces these invalid results? Or that LLVM without your patch miscompiles?
Prior to this patch LLVM will miscompile the `@llvm.nvvm.rotate.right.b64` and `@llvm.nvvm.rotate.b64` intrinsics. The earlier version of this patch expanded the miscompilation to also impact the `rotl` and `rotr` nodes produced from funnel-shift intrinsics. The latest version of the patch fixes all miscompilations.
> I do not see any further changes on the pull requests since I've stamped it last week. Is the issue fixed the this pull request? Or do you want me to help figuring out where things still go wrong?
The further changes are in: 9a0201cc618fb5c2b73fbedd84f6cea9b4638696. After this commit the issues are fixed.
https://github.com/llvm/llvm-project/pull/107655
More information about the llvm-commits
mailing list