[PATCH] D35857: [lit] Fix UnboundLocalError for invalid shtest redirects

Brian Gesiak via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 15:28:25 PDT 2017

modocache added a comment.

@mgorny Thanks for all your help here! Personally I think the best course of action would be to:

1. Mark tests that began failing with https://reviews.llvm.org/D35091 as `XFAIL`
2. Mark tests that fail on Windows as `XFAIL: windows`
3. Un-revert or reimplement https://reviews.llvm.org/rL257221, so that lit tests run continuously
4. Remove the `XFAIL`s one by one

I'd be happy to send the diffs for this tonight or tomorrow, or feel free to beat me to it.



More information about the llvm-commits mailing list