From 87f9dac127b387715d8d96ac7ec8fd469d8c2dab Mon Sep 17 00:00:00 2001 From: Thomas Lively Date: Tue, 3 Dec 2024 11:20:36 -0800 Subject: [NFC] Encapsulate source map reader state (#7132) Move all state relevant to reading source maps out of WasmBinaryReader and into a new utility, SourceMapReader. This is a prerequisite for parallelizing the parsing of function bodies, since the source map reader state is different at the beginning of each function. Also take the opportunity to simplify the way we read source maps, for example by deferring the reading of anything but the position of a debug location until it will be used and by using `std::optional` instead of singleton `std::set`s to store function prologue and epilogue debug locations. --- src/passes/DebugLocationPropagation.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/passes/DebugLocationPropagation.cpp') diff --git a/src/passes/DebugLocationPropagation.cpp b/src/passes/DebugLocationPropagation.cpp index e2d1ac50f..b2eb8fa83 100644 --- a/src/passes/DebugLocationPropagation.cpp +++ b/src/passes/DebugLocationPropagation.cpp @@ -64,10 +64,10 @@ struct DebugLocationPropagation if (auto it = locs.find(previous); it != locs.end()) { locs[curr] = it->second; } - } else if (self->getFunction()->prologLocation.size()) { + } else if (self->getFunction()->prologLocation) { // Instructions may inherit their locations from the function // prolog. - locs[curr] = *self->getFunction()->prologLocation.begin(); + locs[curr] = *self->getFunction()->prologLocation; } } expressionStack.push_back(curr); -- cgit v1.2.3