[PATCH] D30373: [analyzer] NFC: Update test infrastructure to support multiple constraint managers

Devin Coughlin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 2 14:14:07 PST 2017


dcoughlin added a comment.

Thanks for sticking with this!!

This looks good to me, although I would prefer if the definition of the AnalyzerTest class were moved to the bottom of the lit.cfg file. The other configuration and substitutions are far more important to the rest of clang -- people shouldn't have to scan through analyzer-specific stuff to get to configuration of broad relevance.

I also think you may end up needing to conditionalize setting config.analyzer_format to only occur on non-win32 -- but you can commit and see if it triggers the bots.


https://reviews.llvm.org/D30373





More information about the cfe-commits mailing list