[PATCH] D54516: [AMDGPU] Do not mark llvm.amdgcn.set.inactive as IntrNoMem
Nicolai Hähnle via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 16 04:18:50 PST 2018
nhaehnle added a comment.
I agree this needs a test case.
Removing IntrNoMem is not a good solution because it pessimizes other things. Ideally we'd say `convergent` prevents movement across control flow in either direction because other functions are affected by this as well. @sameerds how is that effort going?
Repository:
rL LLVM
https://reviews.llvm.org/D54516
More information about the llvm-commits
mailing list