diff options
| author | Jeremy Morse <jeremy.morse@sony.com> | 2023-12-06 16:36:35 +0000 |
|---|---|---|
| committer | Jeremy Morse <jeremy.morse@sony.com> | 2023-12-06 16:52:10 +0000 |
| commit | 384f916ea899ea6ac9af4a3fb9d0a5b03937acfe (patch) | |
| tree | 5ef3e5bed3e41cd5e93d2df619f05a1ffc642996 /llvm/lib/Transforms/Utils/ModuleUtils.cpp | |
| parent | 565dddec6396d84befa122aa69634b055a60da17 (diff) | |
| download | llvm-384f916ea899ea6ac9af4a3fb9d0a5b03937acfe.zip llvm-384f916ea899ea6ac9af4a3fb9d0a5b03937acfe.tar.gz llvm-384f916ea899ea6ac9af4a3fb9d0a5b03937acfe.tar.bz2 | |
Reapply 34cdc913214fd (#74455), call-site-splitting for RemoveDIs
Original commit message below; asan complained about this commit because it
transpires that the final comparison with CurrentI is in fact a comparison
of a pointer that has been freed. This seems to work fine most of the time,
but using the iterator for such an instruction causes the freed instruction
memory to be accessed, causing a use-after-free. The fix is to perform the
comparison as an instruction, not an iterator.
[NFC][DebugInfo][RemoveDIs] Use iterators to insert in callsite-splitting (#74455)
This patch gets call site splitting to use iterators for insertion
rather than instruction pointers. When we switch on non-instr debug-info
this becomes significant, as the iterators are going to signal whether
or not a position is before or after debug-info.
NFC as this isn't going to affect the output of any existing test.
Diffstat (limited to 'llvm/lib/Transforms/Utils/ModuleUtils.cpp')
0 files changed, 0 insertions, 0 deletions