[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