[PATCH] D40317: [llvm-dwarfdump] Display DW_AT_high_pc as absolute value

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 25 12:14:03 PST 2017


dblaikie added a comment.

In https://reviews.llvm.org/D40317#935075, @dblaikie wrote:

> Happy to hear from Adrian and Paul if this doesn't seem like the right direction - but can be handled post-commit too.
>
> Just the change from getHighAndLow to getHigh, I think.


My comments here were a bit out of date, wrote them before this had been reviewed/signed off by others.

Also before I thought about it and realized getHigh needs the low first, etc...


Repository:
  rL LLVM

https://reviews.llvm.org/D40317





More information about the llvm-commits mailing list