From 95ddd057424b996a0a2a54f282c6c567246d8cc9 Mon Sep 17 00:00:00 2001 From: Alon Zakai Date: Wed, 28 Aug 2024 09:13:33 -0700 Subject: [NFC] Refactor LocalGraph's core getSets API (#6877) Before we just had a map that people would access with localGraph.getSetses[get], while now it is a call localGraph.getSets(get), which more nicely hides the internal implementation details. Also rename getSetses => getSetsMap. This will allow a later PR to optimize the internals of this API. This is performance-neutral as far as I can measure. (We do replace a direct read from a data structure with a call, but the call is in a header and should always get inlined.) --- src/passes/LoopInvariantCodeMotion.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/passes/LoopInvariantCodeMotion.cpp') diff --git a/src/passes/LoopInvariantCodeMotion.cpp b/src/passes/LoopInvariantCodeMotion.cpp index 4239e39c3..321267e8e 100644 --- a/src/passes/LoopInvariantCodeMotion.cpp +++ b/src/passes/LoopInvariantCodeMotion.cpp @@ -228,7 +228,7 @@ struct LoopInvariantCodeMotion bool hasGetDependingOnLoopSet(Expression* curr, LoopSets& loopSets) { FindAll gets(curr); for (auto* get : gets.list) { - auto& sets = localGraph->getSetses[get]; + auto& sets = localGraph->getSets(get); for (auto* set : sets) { // nullptr means a parameter or zero-init value; // no danger to us. -- cgit v1.2.3