[llvm-commits] test-suite: declare i only once in TSVC

Sebastian Pop spop at codeaurora.org
Mon Nov 19 09:56:10 PST 2012


Tobias Grosser wrote:
> On 11/19/2012 06:21 PM, Sebastian Pop wrote:
> >Hi,
> >
> >the attached patch fixes the compile errors when compiling TSVC with -std=gnu89:
> 
> Is there a specific reason why you are running TSVC with --std=gnu89?
> 

No reason.  I just happened to have a script that was setting it and that
crashed my runs of TSVC.

> Would it not make more sense to ensure that the CFLAGS of all test
> programs set the C/C++ mode according to the C/C++ version that is
> used in the test case, rather than working on making all programs
> work with all compilers.

I see that if I do not explicitly set the CFLAGS, TSVC is automatically using
-std=gnu99.

Sebastian
-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation



More information about the llvm-commits mailing list