Re: [clang] 028571d - [clang][Driver] Correct tool search path priority

2020-06-22 Thread David Spickett via cfe-commits
Thanks for the heads up Nico. Yes that was a mistake, it was reviewed as https://reviews.llvm.org/D79842 but somehow I told arc to land it as the duplicate diff instead. I'll look into the failure. On Mon, 22 Jun 2020 at 13:48, Nico Weber wrote: > > Hi, > > https://reviews.llvm.org/D79988 is app

Re: [clang] 028571d - [clang][Driver] Correct tool search path priority

2020-06-22 Thread Nico Weber via cfe-commits
Hi, https://reviews.llvm.org/D79988 is apparently a "Restricted Differential Revision" and I don't have permissions to do that. This being an open source project, this is not something we do. I'm guessing it's not intentional? This also breaks check-clang on macOS: http://45.33.8.238/mac/15950/st

[clang] 028571d - [clang][Driver] Correct tool search path priority

2020-06-22 Thread David Spickett via cfe-commits
Author: David Spickett Date: 2020-06-22T09:41:13+01:00 New Revision: 028571d60843cb87e2637ef69ee09090d4526c62 URL: https://github.com/llvm/llvm-project/commit/028571d60843cb87e2637ef69ee09090d4526c62 DIFF: https://github.com/llvm/llvm-project/commit/028571d60843cb87e2637ef69ee09090d4526c62.diff