[PATCH] Modularize assistant mode

John Thompson john.thompson.jtsoftware at gmail.com
Mon Oct 14 07:16:10 PDT 2013


  This update addresses the comments on the previous revision, except for revising the ensureNoCollisionWithReservedName argument, which I couldn't understand.  Also, I eliminated the ModuleAssistant.h file, merging it with Modularize.h, since it only declared one function.

Hi doug.gregor, silvas,

http://llvm-reviews.chandlerc.com/D1891

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D1891?vs=4837&id=4898#toc

Files:
  test/modularize/Inputs/SubModule1/Header1.h
  test/modularize/Inputs/SubModule1/Header2.h
  test/modularize/Inputs/SubModule2/Header3.h
  test/modularize/Inputs/SubModule2/Header4.h
  test/modularize/NoProblemsAssistant.modularize
  test/modularize/SubModule2.h
  modularize/ModuleAssistant.cpp
  modularize/Modularize.h
  modularize/Modularize.cpp
  modularize/CMakeLists.txt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1891.3.patch
Type: text/x-patch
Size: 19510 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131014/c94a5d21/attachment.bin>


More information about the cfe-commits mailing list