This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Update nop insertion for debugger usage
ClosedPublic

Authored by kzhuravl on May 12 2016, 10:35 AM.

Details

Summary
  • Insert one nop for each high level statement instead of two
  • Do not insert nop before prologue

Diff Detail

Repository
rL LLVM

Event Timeline

kzhuravl updated this revision to Diff 57069.May 12 2016, 10:35 AM
kzhuravl retitled this revision from to [AMDGPU] Update nop insertion for debugger usage.
kzhuravl updated this object.
kzhuravl added reviewers: arsenm, tstellarAMD.
kzhuravl added a subscriber: llvm-commits.
kzhuravl updated this revision to Diff 57219.May 13 2016, 10:41 AM

Update feature description

arsenm accepted this revision.May 13 2016, 10:46 AM
arsenm edited edge metadata.

LGTM

lib/Target/AMDGPU/SIDebuggerInsertNops.cpp
14 ↗(On Diff #57219)

Grammar, such as a debugger?

16 ↗(On Diff #57219)

the debugger

This revision is now accepted and ready to land.May 13 2016, 10:46 AM
kzhuravl marked 2 inline comments as done.May 13 2016, 11:00 AM

Updated grammar

This revision was automatically updated to reflect the committed changes.