[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