This is an archive of the discontinued LLVM Phabricator instance.

[Assignment Tracking][5.1/*] Add deleteAssignmentMarkers function
ClosedPublic

Authored by Orlando on Sep 9 2022, 6:35 AM.

Details

Summary

deleteAssignmentMarkers(const Instruction *Inst) does exactly as you'd expect - it deletes any dbg.assign intrinsics linked to Inst.

I'll fold this into the previous patch (D132224) when landing them, but I wanted to keep the review separate as that one has already been accepted.

Diff Detail

Event Timeline

Orlando created this revision.Sep 9 2022, 6:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 9 2022, 6:35 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
Orlando requested review of this revision.Sep 9 2022, 6:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 9 2022, 6:35 AM

note to self: add unittest

Orlando updated this revision to Diff 459062.Sep 9 2022, 7:44 AM
Orlando retitled this revision from [Assignment Tracing][5.1/*] Add deleteAssignmentMarkers function to [Assignment Tracking][5.1/*] Add deleteAssignmentMarkers function.

+ Add unittest
+ Fix review title

jmorse accepted this revision.Sep 14 2022, 7:38 AM

LGTM with nit

llvm/include/llvm/IR/DebugInfo.h
215
This revision is now accepted and ready to land.Sep 14 2022, 7:38 AM

I'll fold this into the previous patch (D132224) when landing them

As it happens I forgot to do this. I'll be landing these separately.

This revision was landed with ongoing or failed builds.Nov 7 2022, 6:57 AM
This revision was automatically updated to reflect the committed changes.

I am sorry for reverting this patch, but I have to revert it because I need to revert 171f7024cc82e8702abebdedb699d37b50574be7 due to the cyclic dependency that was introduced in the modules build