Szelethus added a comment. In D67420#1668474 <https://reviews.llvm.org/D67420#1668474>, @NoQ wrote:
> It's not that `AnalyzerOptions` is //necessary// to construct this, it's more > like //sufficient//. *sudden enlightenment* Ah, okay, right, I'm a dummie, I think I got what's happening here! :) In that case, the idea sounds great. I'll take a second look on the code later (I have a feeling that we should delete the default ctor of `PathDiagnosticConsumerOptions`, etc), but I'm just a tad too tired atm :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67420/new/ https://reviews.llvm.org/D67420 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits