================ @@ -1986,39 +1986,74 @@ def CIR_BlockAddrDiffAttr } //===----------------------------------------------------------------------===// -// Side Effect +// Memory Effects //===----------------------------------------------------------------------===// -def CIR_SideEffect : CIR_I32Enum< - "SideEffect", "allowed side effects of a function", [ - I32EnumCase<"All", 0, "all">, - I32EnumCase<"Pure", 1, "pure">, - I32EnumCase<"Const", 2, "const"> +def CIR_ModRefInfo : CIR_I32Enum< + "ModRefInfo", "how a function accesses one class of memory", [ + I32EnumCase<"NoModRef", 0, "none">, + I32EnumCase<"Ref", 1, "read">, + I32EnumCase<"Mod", 2, "write">, + I32EnumCase<"ModRef", 3, "readwrite"> ---------------- erichkeane wrote:
should this be a 'bitmask' type of enum? I realize we only have 2 values, but they exactly that :D https://github.com/llvm/llvm-project/pull/223890 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
