This is an archive of the discontinued LLVM Phabricator instance.

[Backend] DBG_CALLSITE & DBG_CALLSITEPARAM instr handling
AbandonedPublic

Authored by djtodoro on Feb 11 2019, 3:45 AM.

Details

Summary

Handle DBG_CALLSITE and DBG_CALLSITEPARAM meta instructions through backend pipeline. It partially relies on the way DBG_VALUE is handled.

Authors: @asowda, @NikolaPrica, @djtodoro, @ivanbaev

Diff Detail

Event Timeline

djtodoro created this revision.Feb 11 2019, 3:45 AM
djtodoro added a project: debug-info.
djtodoro added a subscriber: petarj.
bjope added a subscriber: bjope.Feb 13 2019, 2:40 AM
djtodoro updated this revision to Diff 186612.Feb 13 2019, 2:44 AM
  • Rename: VariableNotChanged ===> ArgumentNotModified
  • Refactor test cases
dstenb added a subscriber: dstenb.Feb 13 2019, 3:06 AM
dstenb added inline comments.
lib/CodeGen/StackColoring.cpp
963

The iterator does seem to be invalidated here (or am I overlooking something?), so you should be able to use BB.instrs() here instead.

lib/CodeGen/StackSlotColoring.cpp
376

MBB.instrs()

dstenb added inline comments.Feb 13 2019, 3:07 AM
lib/CodeGen/StackColoring.cpp
963

Gah, "does _not_".

djtodoro marked 2 inline comments as done.Feb 13 2019, 4:54 AM

@dstenb Thanks for comments!

lib/CodeGen/StackColoring.cpp
963

Yes.

lib/CodeGen/StackSlotColoring.cpp
376

As well as above.

djtodoro updated this revision to Diff 186626.Feb 13 2019, 4:55 AM
  • Use MBB.instrs()
djtodoro abandoned this revision.Apr 15 2019, 8:10 AM