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

[Mehdi AMINI via llvm-commits]

joker.eph added a comment.

What is the semantic of this pointer size? Especially if you have multiple pointer sizes depending on the address space?
Also how does the value returned by MAI::getPointerSize() relates to the DataLayout and/or the TargetMachine getPointerSize()?
The fact that they differs sounds fishy to me.
The fact that the semantic of MAI::getPointerSize() is not well defined is scary to me as well. This looks like something that needs to go away to me, and I'm scared that you want to use it for anything at this point.


Repository:
  rL LLVM

http://reviews.llvm.org/D11969