[flang-commits] [clang] [flang] [llvm] [IR] Convert from nocapture to captures(none) (PR #123181)

[Nikita Popov via flang-commits]

================
@@ -526,6 +526,11 @@ static AttributeSet getIntrinsicArgAttributeSet(LLVMContext &C, unsigned ID) {
 )",
                     ID);
       for (const CodeGenIntrinsic::ArgAttribute &Attr : Attrs) {
+        if (Attr.Kind == CodeGenIntrinsic::NoCapture) {
----------------
nikic wrote:

I'd prefer doing that in a followup, to avoid a mass-change of Intrinsic.td files in this PR. I think we'll also want to expose at least CapturesRetOnly, as some intrinsics want those semantics. Not sure if any intrinsics need something other than CapturesNone/CapturesRetOnly -- exposing the attribute in full generality in TableGen would probably be rather annoying.

https://github.com/llvm/llvm-project/pull/123181