[PATCH] D86622: Fix failing tests after VCTOOLSDIR change

2020-08-26 Thread Alexandre Ganea via Phabricator via cfe-commits
aganea added a comment. I've pushed one more fix: rG33ce275fc156c8b015acfad918937028b2cc235c Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86622/new/ https://reviews.llvm.org/D8

[PATCH] D86622: Fix failing tests after VCTOOLSDIR change

2020-08-26 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. In D86622#2238992 , @zahen wrote: > Sorry again about the break! As with the initial patch, I'll need you to > land this change. No worries, sorry for not catching it in the review. Pushed now as 7a34dca0f3918ab1c0397e56dd64a3c041

[PATCH] D86622: Fix failing tests after VCTOOLSDIR change

2020-08-26 Thread Hans Wennborg via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7a34dca0f391: Fix failing tests after VCTOOLSDIR change (authored by zahen, committed by hans). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D86622: Fix failing tests after VCTOOLSDIR change

2020-08-26 Thread Zachary Henkel via Phabricator via cfe-commits
zahen added a comment. Sorry again about the break! As with the initial patch, I'll need you to land this change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86622/new/ https://reviews.llvm.org/D86622 __

[PATCH] D86622: Fix failing tests after VCTOOLSDIR change

2020-08-26 Thread Zachary Henkel via Phabricator via cfe-commits
zahen created this revision. zahen added a reviewer: hans. Herald added a project: clang. Herald added a subscriber: cfe-commits. zahen requested review of this revision. Switch from hardcoded x64 arch to a regex in the target triple Repository: rG LLVM Github Monorepo https://reviews.llvm.or