[llvm] Extend MemoryEffects to Support Target-Specific Memory Locations (PR #148650)

[Paul Walker via llvm-commits]

================
@@ -656,7 +660,15 @@ std::string Attribute::getAsString(bool InAttrGrp) const {
         break;
       case IRMemLocation::Other:
         llvm_unreachable("This is represented as the default access kind");
+        break;
----------------
paulwalker-arm wrote:

I guess it doesn't matter but you don't need a `break` after an llvm_unreachable.

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