[PATCH] D15495: Macro Support in Dwarf emission

[Amjad Aboud via llvm-commits]

aaboud added a comment.

Thanks Adrian for the comments.
I tried to answer most of them and uploaded a new version with the changes.
Please see my response below.


================
Comment at: lib/CodeGen/AsmPrinter/DwarfDebug.cpp:1919
@@ +1918,3 @@
+  Size += getULEB128Size(F.getLine());
+  DIFile *File = F.getFile();
+  unsigned FID =
----------------
I could not find an elegant and less messy way to do share code between these functions.
Also, in DIE.cpp you can see that there is a similar implementation for "EmitValue()" and "SizeOf()" where common code is not shared!

I tried at least to encapsulate the handling of the macro node array, as it is being repeated three times in the code.

Please, let me know if you have a better idea on how to share the common code above.


http://reviews.llvm.org/D15495