Re: r256216 - [WebAssembly] Remove the -target command-line flag from the ld commandline.

2015-12-23 Thread Dan Gohman via cfe-commits
It appears the reason it was only caught on Windows was that only Windows appends ".exe" to executable files. Your change to match lld with {{.*}} appended fixes it. Thanks! Dan On Tue, Dec 22, 2015 at 2:34 PM, Nico Weber wrote: > This might have broken the test on

Re: r256216 - [WebAssembly] Remove the -target command-line flag from the ld commandline.

2015-12-23 Thread Dan Gohman via cfe-commits
It looks like the two Windows buildbots for clang on that page are already red for other reasons. Dan On Tue, Dec 22, 2015 at 2:54 PM, Nico Weber wrote: > Well yes, but we have a Windows bot on http://lab.llvm.org:8011/console, > right? > > On Tue, Dec 22, 2015 at 5:54

Re: r256216 - [WebAssembly] Remove the -target command-line flag from the ld commandline.

2015-12-22 Thread Nico Weber via cfe-commits
Well yes, but we have a Windows bot on http://lab.llvm.org:8011/console, right? On Tue, Dec 22, 2015 at 5:54 PM, Dan Gohman wrote: > It appears the reason it was only caught on Windows was that only Windows > appends ".exe" to executable files. Your change to match lld with

Re: r256216 - [WebAssembly] Remove the -target command-line flag from the ld commandline.

2015-12-22 Thread Rafael EspĂ­ndola via cfe-commits
Thanks! On 21 December 2015 at 18:30, Dan Gohman via cfe-commits wrote: > Author: djg > Date: Mon Dec 21 17:30:41 2015 > New Revision: 256216 > > URL: http://llvm.org/viewvc/llvm-project?rev=256216=rev > Log: > [WebAssembly] Remove the -target command-line flag from