[cfe-commits] [PATCH] Fix getcontext builtin

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Nov 9 15:10:04 PST 2011


On 4 November 2011 20:04, Dimitry Andric <dimitry at andric.com> wrote:
> On the same note as my previous post, the prototype for the built-in
> function getcontext is incorrect, which leads to warnings like (when
> compiling with -Wsystem-headers):

I think most of the patch is mostly OK. One part that I am not sure is:

+      /// \brief C ucontext_t typedef type
+      SPECIAL_TYPE_ucontext_t                  = 6,
       /// \brief Objective-C "id" redefinition type
-      SPECIAL_TYPE_OBJC_ID_REDEFINITION        = 6,
+      SPECIAL_TYPE_OBJC_ID_REDEFINITION        = 7,
       /// \brief Objective-C "Class" redefinition type
-      SPECIAL_TYPE_OBJC_CLASS_REDEFINITION     = 7,
+      SPECIAL_TYPE_OBJC_CLASS_REDEFINITION     = 8,
       /// \brief Objective-C "SEL" redefinition type
-      SPECIAL_TYPE_OBJC_SEL_REDEFINITION       = 8
+      SPECIAL_TYPE_OBJC_SEL_REDEFINITION       = 9

dgregor, do these numbers have to be stable?

and adding a warning to test/Misc/warning-flags.c is not OK.

Cheers,
Rafael



More information about the cfe-commits mailing list