[llvm] [ThinLTO] optimize propagateAttributes performance (PR #132917)
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 25 09:08:59 PDT 2025
================
@@ -213,9 +211,10 @@ propagateAttributesToRefs(GlobalValueSummary *S,
for (auto &VI : S->refs()) {
assert(VI.getAccessSpecifier() == 0 || isa<FunctionSummary>(S));
if (!VI.getAccessSpecifier()) {
- if (!MarkedNonReadWriteOnly.insert(VI).second)
+ if (VI.getRef()->second.MarkedNonReadWriteOnly)
continue;
- } else if (MarkedNonReadWriteOnly.contains(VI))
----------------
teresajohnson wrote:
I would add an assert in the else case though to confirm this assumption (that MarkedNonReadWriteOnly.contains(VI) is false).
https://github.com/llvm/llvm-project/pull/132917
More information about the llvm-commits
mailing list