[clang] [llvm] [mlir] [LLVM] Improve IR parsing and printing for target memory locations (PR #176968)

[Antonio Frighetto via cfe-commits]

================
@@ -240,6 +240,28 @@ template <typename LocationEnum> class MemoryEffectsBase {
     return getWithoutLoc(Location::InaccessibleMem).doesNotAccessMemory();
   }
 
+  bool isTargetMemLoc(IRMemLocation Loc) {
+    return static_cast<unsigned>(Loc) >=
+           static_cast<unsigned>(Location::FirstTarget);
+  }
+
+  // We need to check if target memory is the same for all locations.
+  // So it behaves the same as default read/write, but for Target
----------------
antoniofrighetto wrote:

Maybe follow the convention with the rest of the code in this header?
```
/// Whether the target memory locations are all the same...
```

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