[PATCH] D11969: Avoid using of DataLayout::getPointerSize() without address space argument in DWARF info generation

[Stanislav Mekhanoshin via llvm-commits]

rampitec added a comment.

> In http://reviews.llvm.org/D11969#223472, @joker.eph wrote:

> 

> > - The DataLayout does not defined anything about which address space the code resides in

> 


I have checked with our engineer doing debug info, and it he confirmed these uses are not related to code addresses, but generally to variable addresses. In fact in our case all code addresses are redirected, so we should not even hit it. Then a bigger type holding pointers is an inefficiency, but not a bug.

I.e. it looks like the name "CodePointerSize" is not correct, a "PointerSize" makes more sense. Maybe a "getLargestPointerSize()" and a "LargestPointerSize" would be better here.
It can even be calculated from the DataLayout itself.


Repository:
  rL LLVM

http://reviews.llvm.org/D11969