Richard, my patch doesn't add support for reference types in C - it only adds
the proper type-checking so that clang emits a meaningful error diagnostic in
this case, instead of crashing with a segfault.
From: [email protected] [mailto:[email protected]] On Behalf Of Richard Smith
Sent: 08 August 2013 00:57
To: Artyom Skrobov
Cc: Anton Korobeynikov; [email protected]
Subject: Re: [PATCH] in C mode, __builtin_va_start tries to accept a reference,
sees a record, decides it's a C++ class, then crashes because it isn't
This is truly hideous. Do we *really* want to support reference types in C,
just so that we can handle __builtin_va_start?
On Tue, Aug 6, 2013 at 8:26 AM, Artyom Skrobov
<[email protected]<mailto:[email protected]>> wrote:
Anton, thank you for your suggestion.
Re-submitting the patch as an attachment.
-----Original Message-----
From: Anton Korobeynikov
[mailto:[email protected]<mailto:[email protected]>]
Sent: 05 August 2013 23:24
To: Artyom Skrobov
Cc: [email protected]<mailto:[email protected]>
Subject: Re: [PATCH] in C mode, __builtin_va_start tries to accept a reference,
sees a record, decides it's a C++ class, then crashes because it isn't
Please consider reading http://llvm.org/docs/DeveloperPolicy.html#making-a-patch
-- IMPORTANT NOTICE: The contents of this email and any attachments are
confidential and may also be privileged. If you are not the intended recipient,
please notify the sender immediately and do not disclose the contents to any
other person, use it for any purpose, or store or copy the information in any
medium. Thank you.
_______________________________________________
cfe-commits mailing list
[email protected]<mailto:[email protected]>
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
-- IMPORTANT NOTICE: The contents of this email and any attachments are
confidential and may also be privileged. If you are not the intended recipient,
please notify the sender immediately and do not disclose the contents to any
other person, use it for any purpose, or store or copy the information in any
medium. Thank you.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
