[PATCH] This change-list changes clang to attach the ordinary comments if an option is specified
Amin Shali
amshali at gmail.com
Wed Apr 3 15:55:18 PDT 2013
Changes in this revision:
* Applied reviewer's comments.
* Renamed the option to ParseAllComments.
* Fixed the code appropriately.
* Removed the test from unittests/Frontend/FrontendActionTest.cpp. * Added two tests, one for making sure we are passing the argument to clang and one for making sure we are attaching ordinary comments.
* Changed Tools.cpp to pass the option to cc1.
* Changed the option definition in Options.td.
Hi chandlerc, ddunbar,
http://llvm-reviews.chandlerc.com/D614
CHANGE SINCE LAST DIFF
http://llvm-reviews.chandlerc.com/D614?vs=1486&id=1500#toc
Files:
include/clang/AST/RawCommentList.h
include/clang/Basic/CommentOptions.h
include/clang/Driver/Options.td
lib/AST/ASTContext.cpp
lib/AST/RawCommentList.cpp
lib/Driver/Tools.cpp
lib/Frontend/CompilerInvocation.cpp
lib/Sema/Sema.cpp
lib/Serialization/ASTReader.cpp
test/Driver/fparse-all-comments.c
test/Index/parse-all-comments.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D614.2.patch
Type: text/x-patch
Size: 10032 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130403/1f8605f0/attachment.bin>
More information about the cfe-commits
mailing list