[PATCH] D141854: [Assignment Tracking] Fix invalidated iterator usage
Orlando Cazalet-Hyams via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 16 07:34:06 PST 2023
Orlando created this revision.
Orlando added reviewers: StephenTozer, jmorse.
Orlando added a project: debug-info.
Herald added a subscriber: hiraditya.
Herald added a project: All.
Orlando requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
The iterator `FirstOverlap` is invalidated after the call to `insert` - avoid dereferencing the iterator after the call to `insert`.
https://reviews.llvm.org/D141854
Files:
llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
Index: llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
===================================================================
--- llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
+++ llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
@@ -651,14 +651,19 @@
// [ - i - ]
// +
// [ i ][ f ][ i ]
+
+ // Save values for use after inserting a new interval.
auto EndBitOfOverlap = FirstOverlap.stop();
+ unsigned OverlapValue = FirstOverlap.value();
+
+ // Shorten the overlapping interval.
FirstOverlap.setStop(StartBit);
insertMemLoc(BB, Before, Var, FirstOverlap.start(), StartBit,
- *FirstOverlap, VarLoc.DL);
+ OverlapValue, VarLoc.DL);
// Insert a new interval to represent the end part.
- FragMap.insert(EndBit, EndBitOfOverlap, *FirstOverlap);
- insertMemLoc(BB, Before, Var, EndBit, EndBitOfOverlap, *FirstOverlap,
+ FragMap.insert(EndBit, EndBitOfOverlap, OverlapValue);
+ insertMemLoc(BB, Before, Var, EndBit, EndBitOfOverlap, OverlapValue,
VarLoc.DL);
// Insert the new (middle) fragment now there is space.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D141854.489557.patch
Type: text/x-patch
Size: 1164 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230116/38375e27/attachment.bin>
More information about the llvm-commits
mailing list