[PATCH] D16135: Macro Debug Info support in Clang

Amjad Aboud via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 29 14:13:38 PST 2016


aaboud updated this revision to Diff 82696.
aaboud added a comment.
Herald added subscribers: mgorny, nemanjai.

Improved code based on Richard's comments.
Removed the change to ASTConsumer, now it does not need to know anything about PP consumer.
However, I still needed to change the CodeGenerator ASTConsumer to return the CGDebugInfo class.

Richard, please review the new patch.
This time I made sure to add the "MacroPPCallbacks.*" new files.


https://reviews.llvm.org/D16135

Files:
  include/clang/CodeGen/ModuleBuilder.h
  lib/CodeGen/CGDebugInfo.cpp
  lib/CodeGen/CGDebugInfo.h
  lib/CodeGen/CMakeLists.txt
  lib/CodeGen/CodeGenAction.cpp
  lib/CodeGen/MacroPPCallbacks.cpp
  lib/CodeGen/MacroPPCallbacks.h
  lib/CodeGen/ModuleBuilder.cpp
  test/CodeGen/debug-info-global-constant.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16135.82696.patch
Type: text/x-patch
Size: 18747 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161229/8d4b6ccd/attachment.bin>


More information about the cfe-commits mailing list