[llvm] [DirectX][NFC] Change specification of overload types and attribute in DXIL.td (PR #81184)

S. Bharadwaj Yadavalli via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 8 15:46:01 PST 2024


================
@@ -268,36 +296,45 @@ static void emitDXILIntrinsicMap(std::vector<DXILOperationDesc> &Ops,
 
 static std::string emitDXILOperationFnAttr(StringRef FnAttr) {
   return StringSwitch<std::string>(FnAttr)
-      .Case("rn", "Attribute::ReadNone")
-      .Case("ro", "Attribute::ReadOnly")
+      .Case("NoReadMemory", "Attribute::ReadNone")
+      .Case("ReadMemory", "Attribute::ReadOnly")
----------------
bharadwajy wrote:

> ReadMemory lost the only part of ReadOnly, but ReadOnlyMemory feels bad too :( Maybe OnlyReadMemory?

How about `MemoryReadNone` and `MemoryReadOnly`?

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


More information about the llvm-commits mailing list