[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/llvm-commits/attachments/20130403/1f8605f0/attachment.bin>


More information about the llvm-commits mailing list