[PATCH] D16697: Updating .debug_line section version information to match DWARF version.

Katya Romanova via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 10 15:11:58 PDT 2016


kromanova added a comment.

>> Question: If you grep LLVM sources for " DWARF2Line, you'll see that a lot of identifier names imply that line_table is DWARF2. e.g. "DWARF2LineOpcodeBase". I suspect this should be changed to avoid confusion. Do you agree? Separate NFC patch?
> 
> Seems reasonable to me but I would generally expect that kind of name-change patch to be done separately.

That sounds good. I will do it after this patch is accepted. I would really like to get this patch reviewed (and tested) by someone who cares about VLIW.


Repository:
  rL LLVM

https://reviews.llvm.org/D16697





More information about the llvm-commits mailing list