This is an archive of the discontinued LLVM Phabricator instance.

Move VariableDbgInfo from MachineModuleInfo to MachineFunction
ClosedPublic

Authored by MatzeB on Nov 28 2016, 6:24 PM.

Details

Summary

VariableDbgInfo is per function data, so it makes sense to have it with
the function instead of the module.

This is a necessary step to have machine module passes work properly.

Diff Detail

Repository
rL LLVM

Event Timeline

MatzeB updated this revision to Diff 79495.Nov 28 2016, 6:24 PM
MatzeB retitled this revision from to Move VariableDbgInfo from MachineModuleInfo to MachineFunction.
MatzeB updated this object.
MatzeB added reviewers: echristo, aprantl.
MatzeB set the repository for this revision to rL LLVM.
MatzeB added a subscriber: llvm-commits.
MatzeB updated this revision to Diff 79631.Nov 29 2016, 1:36 PM

Slightly simplify patch by not adding const MachineFunction& parameters to some AsmPrinter functions. The current function is accessible by Asm->MF.

MatzeB updated this revision to Diff 79635.Nov 29 2016, 1:47 PM

Upload correctly updated patch this time :)

echristo accepted this revision.Nov 30 2016, 3:19 PM
echristo edited edge metadata.

LGTM.

-eric

This revision is now accepted and ready to land.Nov 30 2016, 3:19 PM
This revision was automatically updated to reflect the committed changes.