Patch: Change from configure time to compile time probe for intptr_t, definition.
Mon Mar 23 14:19:00 GMT 2015
On Mar 23 08:12, Joel Sherrill wrote:
> On 03/23/2015 05:17 AM, Corinna Vinschen wrote:
> >Hi Joel,
> >The patch looks good... almost. No worries, it's my fault:
> Great. This patch seems to take care of the issue pretty
> I spent a lot of time building over the past few days and this
> patch really helped reduce the warnings in the RTEMS tests.
> We are almost to 0 warnings across the targets except m32c
> which ICEs so much building RTEMS with "make -k" that I
> can't tell much of anything.
> >It occured to me that this is not without danger. Assuming you're using
> >a compiler ignoring the push_macro/pop_macro pragmas, then the above
> >macro definitions of signed, int and long will prevail from this point
> >on. Therefore, to be really sure, another set of #undef's should be
> >inserted here:
> > #undef signed
> > #undef int
> > #undef long
> OK. I will update the patch.
> I also noticed that newlib.hin didn't get updated. What's the
> magic for that?
> >>+#pragma pop_macro("signed")
> >>+#pragma pop_macro("int")
> >>+#pragma pop_macro("long")
> >> #endif /* __SYS_CONFIG_H__ */
> >With that change, ok to push.
> My first newlib git push. :)
I was nervous like hell :)
> Is there any guidance for the commit message? I think binutils/gdb
> has been doing something like this:
> summary line/title
> ChangeLog entry
> Is that OK?
That's what I'm doing as well, except for very tiny stuff for which
I just use the summary line. It certainly won't hurt to follow
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 819 bytes
Desc: not available
More information about the Newlib