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

Dominic Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 27 19:36:22 PST 2017


ddcc reopened this revision.
ddcc added a comment.
This revision is now accepted and ready to land.

It looks like there's some sort of pickling-related test failure when lit tries to fork, because it can't find the class `lit.TestingConfig.AnalyzerTest`: http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/6154/steps/test/logs/stdio . I'm guessing that this has something to do with the `AnalyzerTest` class being loaded dynamically from `lit.local.cfg` , but short of moving it out into `llvm/utils/lit/lit/formats`, I have no idea how to fix this. Also, I can't reproduce this problem locally with `ninja -j4 check-clang`. @dcoughlin ydo you have any suggestions?


Repository:
  rL LLVM

https://reviews.llvm.org/D30373





More information about the cfe-commits mailing list