Sisyph wrote: > The patch to SemaExpr looks reasonable to me. I'd suggest that goes in > separate from the amdgpu intrinsic stuff. > > I'd test this by tweaking the code to do the current lowering _and_ the > proposed and check that they do exactly the same thing on all the existing > builtins, then drop the current code path, but ymmv in terms of how that > strategy interacts with our code review system.
Thanks. I will wait for https://github.com/llvm/llvm-project/pull/137425 to be resolved, then rebase or convert this PR as needed. https://github.com/llvm/llvm-project/pull/138141 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits