[cfe-commits] [PATCH] Force C89 for c-index-test.c

Dmitri Gribenko gribozavr at gmail.com
Sat Sep 22 11:49:06 PDT 2012


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.

Please review.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: c-index-test-c89-v1.patch
Type: application/octet-stream
Size: 397 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120922/0d5ca3e6/attachment.obj>


More information about the cfe-commits mailing list