[cfe-commits] [PATCH] Force C89 for c-index-test.c
    Douglas Gregor 
    dgregor at apple.com
       
    Sun Sep 30 21:32:22 PDT 2012
    
    
  
On Sep 22, 2012, at 11:49 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> Hello,
> 
> MSVC supports only C89, so it is important to keep c-index-test.c
> buildable with C89.  However, Clang defaults to C99, so while building
> Clang with Clang one can introduce C99 constructs into c-index-test.c
> without noticing.
> 
> The attached patch forces C89 for c-index-test.c in CMake.
> Unfortunately, my knowledge of Makefile-based build system is not
> sufficient to make an equivalent modification.
LGTM.
	- Doug
    
    
More information about the cfe-commits
mailing list