Re: [PATCH] D17959: Make it possible for AST plugins to enable themselves by default

2016-03-08 Thread John Brawn via cfe-commits
john.brawn added a comment. Looking at http://reviews.llvm.org/D15729 I //think// there's no conflict between what this patch is doing and what that patch is doing. If a plugin returns AddBeforeMainAction or AddAfterMainAction then when ASTUnit in libclang loads that plugin it should get

Re: [PATCH] D17959: Make it possible for AST plugins to enable themselves by default

2016-03-08 Thread Manuel Klimek via cfe-commits
klimek added a reviewer: bkramer. klimek added a comment. +benjamin, who has done work in the area Repository: rL LLVM http://reviews.llvm.org/D17959 ___ cfe-commits mailing list cfe-commits@lists.llvm.org