[cfe-commits] [LLVMdev] clang errors on void main()

Vasiliy Korchagin korchagin at ispras.ru
Tue Feb 14 07:10:48 PST 2012

14.02.2012 05:20, John McCall пишет:
> On Feb 13, 2012, at 4:38 PM, John McCall wrote:
>> On Feb 11, 2012, at 7:31 AM, Vasiliy Korchagin wrote:
>>> I agree, without setting implicit return zero bit changes in codegen are not necessary. New version of patch is attached.
>> This check has nothing to do with C99.  You should be checking for "has GNU extensions" and "is not C++", i.e.
>>   getLangOptions().GNU&&  !getLangOptions().CPlusPlus
> I see that I missed the later follow-ups in this thread, and you're now
> checking CPlusPlus correctly.  Please do also check for GNU
> extensions being enabled and implement this:
>> Also, please have CheckMain just set hasImplicitReturnZero() as appropriate and then make actOnFinishFunctionBody disable fall-through checking when that bit is set.
> With that, this should be fine.
> John.
I fixed the patch. New patch is attached.

Vasiliy Korchagin,
The Institute for System Programming (ISP) of the Russian Academy of 
Sciences (RAS)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: allow-non-int-main-option.patch
Type: text/x-patch
Size: 3603 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120214/7d6bb62e/attachment.bin>

More information about the cfe-commits mailing list