saugustine added a subscriber: klimek. saugustine added a comment. In https://reviews.llvm.org/D34304#783675, @klimek wrote:
> I think a better way might be to generally leave dependency options alone, > add a default argument adapter to filter out all deps related flags, and > allow users to add their own argument adapters that don't do that. This argument adapter would have to be passed down in a similar way, no? buildASTFromCodeWithArgs, toolIinvocation::Run, and newInvocation are all entry-points that would need this behavior, and all are called by themselves in one place or another. I'm happy to do that, as it is quite a bit more flexible, but it doesn't seem any cleaner. https://reviews.llvm.org/D34304 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
