[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