[llvm] [IR] Memory locations for floating-point operations (PR #226826)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 1 04:08:17 PDT 2026
================
@@ -2860,7 +2865,7 @@ std::optional<MemoryEffects> LLParser::parseMemoryAttr() {
tokError("default access kind must be specified first");
return std::nullopt;
}
- ME = MemoryEffects(*MR);
+ ME = MemoryEffects(*MR, /*NoFPEnvMarker*/ false);
----------------
CarolineConcatto wrote:
This set Inaccessible memory, but does not set fp memory locations.
It is a bit strange that sometime FP memory location is equal to inaccessible memory(onlyAccessesInaccessibleMem or in getModRefInfoInaccessibleAndTargetMemLoc ) and sometimes is not.
Like now, FP locations is not being treated as inaccessible memory, because this sets all locations including inaccessible memory.
Maybe it would be better to differentiate them, like we did for target_memory.
https://github.com/llvm/llvm-project/pull/226826
More information about the llvm-commits
mailing list