Re: r347720 - [RISCV] Mark unit tests as "requires: riscv-registered-target"

2018-12-05 Thread Grang, Mandeep Singh via cfe-commits
Hi Alex, Sorry, I missed this thread. I guess I was too quick to commit my patch. The actual problem seems to be something else due to which riscv32-toolchain.c fails on Windows. The fact that upstream bots seem to be fine could point to a bug in our internal code. Basically it is not able

Re: r347720 - [RISCV] Mark unit tests as "requires: riscv-registered-target"

2018-12-05 Thread Alex Bradbury via cfe-commits
On Thu, 29 Nov 2018 at 09:44, Alex Bradbury wrote: > > On Tue, 27 Nov 2018 at 22:56, Mandeep Singh Grang via cfe-commits > wrote: > > > > Author: mgrang > > Date: Tue Nov 27 14:53:57 2018 > > New Revision: 347720 > > > > URL: http://llvm.org/viewvc/llvm-project?rev=347720=rev > > Log: > >

Re: r347720 - [RISCV] Mark unit tests as "requires: riscv-registered-target"

2018-11-29 Thread Alex Bradbury via cfe-commits
On Tue, 27 Nov 2018 at 22:56, Mandeep Singh Grang via cfe-commits wrote: > > Author: mgrang > Date: Tue Nov 27 14:53:57 2018 > New Revision: 347720 > > URL: http://llvm.org/viewvc/llvm-project?rev=347720=rev > Log: > [RISCV] Mark unit tests as "requires: riscv-registered-target" > > Some of these

r347720 - [RISCV] Mark unit tests as "requires: riscv-registered-target"

2018-11-27 Thread Mandeep Singh Grang via cfe-commits
Author: mgrang Date: Tue Nov 27 14:53:57 2018 New Revision: 347720 URL: http://llvm.org/viewvc/llvm-project?rev=347720=rev Log: [RISCV] Mark unit tests as "requires: riscv-registered-target" Some of these tests break if the RISCV backend has not been built. Reland D54816. Modified: