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

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 24 10:11:15 PDT 2018


aprantl added a comment.

> But it uses OrderMap.Orders[&DbgVal] to get the right location in the instructions.

Do you have OrderMap readily available or would you 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.


https://reviews.llvm.org/D50887





More information about the llvm-commits mailing list