[PATCH] D50887: [DWARF] Missing location debug information with -O2.

[Carlos Alberto Enciso via Phabricator via llvm-commits]

CarlosAlbertoEnciso marked an inline comment as done.
CarlosAlbertoEnciso added a comment.

In https://reviews.llvm.org/D50887#1212565, @aprantl wrote:

> Do you have OrderMap readily available or would you need to compute it first?


At the point where 'collectDebugValues' is called, OrderMap is not available. I need to compute it first.

>> If the goal is to make 'collectDebugValues' more generic, that function should be moved to MachineRegisterInfo and have the steps (1), (2) and (3).
> 
> That doesn't sound like a bad idea.

That would be a good improvement.


https://reviews.llvm.org/D50887