diff options
| author | Alon Zakai <azakai@google.com> | 2024-08-28 09:13:33 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-08-28 09:13:33 -0700 |
| commit | 95ddd057424b996a0a2a54f282c6c567246d8cc9 (patch) | |
| tree | 5fb5cf34c779f1cf9cb49208e470d557db91b177 /src/wasm/wasm-stack-opts.cpp | |
| parent | e2ceaa58c10e9ee3e9eece42466243f5a8aff125 (diff) | |
| download | binaryen-95ddd057424b996a0a2a54f282c6c567246d8cc9.tar.gz binaryen-95ddd057424b996a0a2a54f282c6c567246d8cc9.tar.bz2 binaryen-95ddd057424b996a0a2a54f282c6c567246d8cc9.zip | |
[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.)
Diffstat (limited to 'src/wasm/wasm-stack-opts.cpp')
| -rw-r--r-- | src/wasm/wasm-stack-opts.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/wasm/wasm-stack-opts.cpp b/src/wasm/wasm-stack-opts.cpp index cf4c094b5..5902b40a5 100644 --- a/src/wasm/wasm-stack-opts.cpp +++ b/src/wasm/wasm-stack-opts.cpp @@ -225,7 +225,7 @@ void StackIROptimizer::local2Stack() { if (set->index == get->index) { // This might be a proper set-get pair, where the set is // used by this get and nothing else, check that. - auto& sets = localGraph.getSetses[get]; + auto& sets = localGraph.getSets(get); if (sets.size() == 1 && *sets.begin() == set) { auto& setInfluences = localGraph.setInfluences[set]; // If this has the proper value of 1, also do the potentially- |