Re: r258772 - Revert the test for r258720 temporarily

2016-01-25 Thread Ehsan Akhgari via cfe-commits
Thanks for the suggestion! I converted the test to a -### test and recommitted. Also rewrote the existing test (which I guess was failing for another reason than what the author intended since it was merely checking for an error return code...) On Mon, Jan 25, 2016 at 9:16 PM, Nico Weber wrote:

Re: r258772 - Revert the test for r258720 temporarily

2016-01-25 Thread Nico Weber via cfe-commits
Since it isn't a -### test, the test probably just needs a `// REQUIRES: x86-registered-target`? Is adding -### a possibility? On Mon, Jan 25, 2016 at 8:51 PM, Ehsan Akhgari via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: ehsan > Date: Mon Jan 25 19:51:47 2016 > New Revision: 25877

r258772 - Revert the test for r258720 temporarily

2016-01-25 Thread Ehsan Akhgari via cfe-commits
Author: ehsan Date: Mon Jan 25 19:51:47 2016 New Revision: 258772 URL: http://llvm.org/viewvc/llvm-project?rev=258772&view=rev Log: Revert the test for r258720 temporarily This test is failing on a bot for reasons that are unclear to me. Reverting for now... Modified: cfe/trunk/test/Driver/u