* [PATCH] nios2-linux: add missing cpp specs @ 2015-05-27 16:05 Mike Frysinger 2015-05-29 15:41 ` Sandra Loosemore 0 siblings, 1 reply; 7+ messages in thread From: Mike Frysinger @ 2015-05-27 16:05 UTC (permalink / raw) To: gcc-patches; +Cc: cltang, sandra Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work like on all other linux targets. 2015-05-27 Mike Frysinger <vapier@gentoo.org> * config/nios2/linux.h (CPP_SPEC): Define. --- gcc/config/nios2/linux.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/gcc/config/nios2/linux.h b/gcc/config/nios2/linux.h index 41cad94..f43f655 100644 --- a/gcc/config/nios2/linux.h +++ b/gcc/config/nios2/linux.h @@ -26,6 +26,9 @@ } \ while (0) +#undef CPP_SPEC +#define CPP_SPEC "%{posix:-D_POSIX_SOURCE} %{pthread:-D_REENTRANT}" + #define GLIBC_DYNAMIC_LINKER "/lib/ld-linux-nios2.so.1" #undef LINK_SPEC -- 2.4.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nios2-linux: add missing cpp specs 2015-05-27 16:05 [PATCH] nios2-linux: add missing cpp specs Mike Frysinger @ 2015-05-29 15:41 ` Sandra Loosemore 2015-05-29 18:42 ` Mike Frysinger 0 siblings, 1 reply; 7+ messages in thread From: Sandra Loosemore @ 2015-05-29 15:41 UTC (permalink / raw) To: Mike Frysinger; +Cc: gcc-patches, cltang On 05/27/2015 10:00 AM, Mike Frysinger wrote: > Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work > like on all other linux targets. > > 2015-05-27 Mike Frysinger <vapier@gentoo.org> > > * config/nios2/linux.h (CPP_SPEC): Define. I see that -posix is not documented at all in invoke.texi and -pthread is documented only for RS6000 and Solaris (which is not Linux). What, exactly, are these options supposed to do on "all other linux targets"? If these options are supposed to be generic to all Linux targets, can't they be handled in some common way instead of duplicating the CPP_SPEC code in all the individual back ends? -Sandra ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nios2-linux: add missing cpp specs 2015-05-29 15:41 ` Sandra Loosemore @ 2015-05-29 18:42 ` Mike Frysinger 2015-05-29 19:32 ` Sandra Loosemore 0 siblings, 1 reply; 7+ messages in thread From: Mike Frysinger @ 2015-05-29 18:42 UTC (permalink / raw) To: Sandra Loosemore; +Cc: gcc-patches, cltang [-- Attachment #1: Type: text/plain, Size: 792 bytes --] On 29 May 2015 08:44, Sandra Loosemore wrote: > On 05/27/2015 10:00 AM, Mike Frysinger wrote: > > Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work > > like on all other linux targets. > > > > 2015-05-27 Mike Frysinger <vapier@gentoo.org> > > > > * config/nios2/linux.h (CPP_SPEC): Define. > > I see that -posix is not documented at all in invoke.texi and -pthread > is documented only for RS6000 and Solaris (which is not Linux). What, > exactly, are these options supposed to do on "all other linux targets"? > If these options are supposed to be generic to all Linux targets, > can't they be handled in some common way instead of duplicating the > CPP_SPEC code in all the individual back ends? please see my other threads/patches -mike [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nios2-linux: add missing cpp specs 2015-05-29 18:42 ` Mike Frysinger @ 2015-05-29 19:32 ` Sandra Loosemore 2015-05-30 4:50 ` Mike Frysinger 0 siblings, 1 reply; 7+ messages in thread From: Sandra Loosemore @ 2015-05-29 19:32 UTC (permalink / raw) To: Mike Frysinger; +Cc: gcc-patches, cltang On 05/29/2015 11:36 AM, Mike Frysinger wrote: > On 29 May 2015 08:44, Sandra Loosemore wrote: >> On 05/27/2015 10:00 AM, Mike Frysinger wrote: >>> Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work >>> like on all other linux targets. >>> >>> 2015-05-27 Mike Frysinger <vapier@gentoo.org> >>> >>> * config/nios2/linux.h (CPP_SPEC): Define. >> >> I see that -posix is not documented at all in invoke.texi and -pthread >> is documented only for RS6000 and Solaris (which is not Linux). What, >> exactly, are these options supposed to do on "all other linux targets"? >> If these options are supposed to be generic to all Linux targets, >> can't they be handled in some common way instead of duplicating the >> CPP_SPEC code in all the individual back ends? > > please see my other threads/patches (Sorry, I am a few days behind in mailing list traffic, was just trying to respond to the review request that was CC'ed to me directly.) Do you mean this one? https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02708.html That addresses my concern about not duplicating this in every back end, but I still don't see any documentation and don't really understand what these flags are supposed to do or why I might need to add them to my command line. Taking off my nios2 maintainer hat and putting on the docs maintainer one instead, I think proper documentation for these options is a requirement here.... -Sandra ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nios2-linux: add missing cpp specs 2015-05-29 19:32 ` Sandra Loosemore @ 2015-05-30 4:50 ` Mike Frysinger 2015-05-30 16:09 ` Sandra Loosemore 0 siblings, 1 reply; 7+ messages in thread From: Mike Frysinger @ 2015-05-30 4:50 UTC (permalink / raw) To: Sandra Loosemore; +Cc: gcc-patches, cltang [-- Attachment #1: Type: text/plain, Size: 1743 bytes --] On 29 May 2015 12:32, Sandra Loosemore wrote: > On 05/29/2015 11:36 AM, Mike Frysinger wrote: > > On 29 May 2015 08:44, Sandra Loosemore wrote: > >> On 05/27/2015 10:00 AM, Mike Frysinger wrote: > >>> Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work > >>> like on all other linux targets. > >>> > >>> 2015-05-27 Mike Frysinger <vapier@gentoo.org> > >>> > >>> * config/nios2/linux.h (CPP_SPEC): Define. > >> > >> I see that -posix is not documented at all in invoke.texi and -pthread > >> is documented only for RS6000 and Solaris (which is not Linux). What, > >> exactly, are these options supposed to do on "all other linux targets"? > >> If these options are supposed to be generic to all Linux targets, > >> can't they be handled in some common way instead of duplicating the > >> CPP_SPEC code in all the individual back ends? > > > > please see my other threads/patches > > (Sorry, I am a few days behind in mailing list traffic, was just trying > to respond to the review request that was CC'ed to me directly.) > > Do you mean this one? > > https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02708.html yes > That addresses my concern about not duplicating this in every back end, > but I still don't see any documentation and don't really understand what > these flags are supposed to do or why I might need to add them to my > command line. Taking off my nios2 maintainer hat and putting on the > docs maintainer one instead, I think proper documentation for these > options is a requirement here.... i'm not familiar with the history. i'm merely cleaning up some of the mess. both defines are respected by glibc and make a difference to compilation. -mike [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nios2-linux: add missing cpp specs 2015-05-30 4:50 ` Mike Frysinger @ 2015-05-30 16:09 ` Sandra Loosemore 2015-05-30 19:13 ` Mike Frysinger 0 siblings, 1 reply; 7+ messages in thread From: Sandra Loosemore @ 2015-05-30 16:09 UTC (permalink / raw) To: Mike Frysinger; +Cc: gcc-patches, cltang On 05/29/2015 09:28 PM, Mike Frysinger wrote: > On 29 May 2015 12:32, Sandra Loosemore wrote: >> On 05/29/2015 11:36 AM, Mike Frysinger wrote: >>> On 29 May 2015 08:44, Sandra Loosemore wrote: >>>> On 05/27/2015 10:00 AM, Mike Frysinger wrote: >>>>> Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work >>>>> like on all other linux targets. >>>>> >>>>> 2015-05-27 Mike Frysinger <vapier@gentoo.org> >>>>> >>>>> * config/nios2/linux.h (CPP_SPEC): Define. >>>> >>>> I see that -posix is not documented at all in invoke.texi and -pthread >>>> is documented only for RS6000 and Solaris (which is not Linux). What, >>>> exactly, are these options supposed to do on "all other linux targets"? >>>> If these options are supposed to be generic to all Linux targets, >>>> can't they be handled in some common way instead of duplicating the >>>> CPP_SPEC code in all the individual back ends? >>> >>> please see my other threads/patches >> >> (Sorry, I am a few days behind in mailing list traffic, was just trying >> to respond to the review request that was CC'ed to me directly.) >> >> Do you mean this one? >> >> https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02708.html > > yes > >> That addresses my concern about not duplicating this in every back end, >> but I still don't see any documentation and don't really understand what >> these flags are supposed to do or why I might need to add them to my >> command line. Taking off my nios2 maintainer hat and putting on the >> docs maintainer one instead, I think proper documentation for these >> options is a requirement here.... > > i'm not familiar with the history. i'm merely cleaning up some of the mess. > both defines are respected by glibc and make a difference to compilation. I see that the glibc manual does document the preprocessor macros, but it doesn't look like glibc either uses or documents the use of special GCC command-line options, so I am still confused about why we need these options. However, if using these command-line options on targets where they previously have existed is common practice, I have no real objection to adding them to nios2 as well. But, please note that the GCC coding conventions https://gcc.gnu.org/codingconventions.html#Documentation explicitly requires documentation for all command-line options. So, if you are cleaning up the mess to generalize the implementation of these options so that they are no longer confined to a few specific targets, you must also move the documentation for these options out of the back-end-specific part of the manual and make sure it accurately reflects the current purpose and usage of these options. -Sandra ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nios2-linux: add missing cpp specs 2015-05-30 16:09 ` Sandra Loosemore @ 2015-05-30 19:13 ` Mike Frysinger 0 siblings, 0 replies; 7+ messages in thread From: Mike Frysinger @ 2015-05-30 19:13 UTC (permalink / raw) To: Sandra Loosemore; +Cc: gcc-patches, cltang [-- Attachment #1: Type: text/plain, Size: 3378 bytes --] On 30 May 2015 09:38, Sandra Loosemore wrote: > On 05/29/2015 09:28 PM, Mike Frysinger wrote: > > On 29 May 2015 12:32, Sandra Loosemore wrote: > >> On 05/29/2015 11:36 AM, Mike Frysinger wrote: > >>> On 29 May 2015 08:44, Sandra Loosemore wrote: > >>>> On 05/27/2015 10:00 AM, Mike Frysinger wrote: > >>>>> Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work > >>>>> like on all other linux targets. > >>>>> > >>>>> 2015-05-27 Mike Frysinger <vapier@gentoo.org> > >>>>> > >>>>> * config/nios2/linux.h (CPP_SPEC): Define. > >>>> > >>>> I see that -posix is not documented at all in invoke.texi and -pthread > >>>> is documented only for RS6000 and Solaris (which is not Linux). What, > >>>> exactly, are these options supposed to do on "all other linux targets"? > >>>> If these options are supposed to be generic to all Linux targets, > >>>> can't they be handled in some common way instead of duplicating the > >>>> CPP_SPEC code in all the individual back ends? > >>> > >>> please see my other threads/patches > >> > >> (Sorry, I am a few days behind in mailing list traffic, was just trying > >> to respond to the review request that was CC'ed to me directly.) > >> > >> Do you mean this one? > >> > >> https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02708.html > > > > yes > > > >> That addresses my concern about not duplicating this in every back end, > >> but I still don't see any documentation and don't really understand what > >> these flags are supposed to do or why I might need to add them to my > >> command line. Taking off my nios2 maintainer hat and putting on the > >> docs maintainer one instead, I think proper documentation for these > >> options is a requirement here.... > > > > i'm not familiar with the history. i'm merely cleaning up some of the mess. > > both defines are respected by glibc and make a difference to compilation. > > I see that the glibc manual does document the preprocessor macros, but > it doesn't look like glibc either uses or documents the use of special > GCC command-line options, so I am still confused about why we need these > options. However, if using these command-line options on targets where > they previously have existed is common practice, I have no real > objection to adding them to nios2 as well. > > But, please note that the GCC coding conventions > > https://gcc.gnu.org/codingconventions.html#Documentation > > explicitly requires documentation for all command-line options. > > So, if you are cleaning up the mess to generalize the implementation of > these options so that they are no longer confined to a few specific > targets, you must also move the documentation for these options out of > the back-end-specific part of the manual and make sure it accurately > reflects the current purpose and usage of these options. it's not that i disagree with anything you're saying wrt documentation -- the situation is pretty bad. it's that i think it's not directly impacted by my changes. my cleanups also do not really move them out of the back end ... they still only work for gnu/linux type targets. so while we should see about fixing the docs, i don't think my patch should depend on that considering (1) it's an obvious (imo) improvement and (2) the docs have always been broken. -mike [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-05-30 16:24 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-05-27 16:05 [PATCH] nios2-linux: add missing cpp specs Mike Frysinger 2015-05-29 15:41 ` Sandra Loosemore 2015-05-29 18:42 ` Mike Frysinger 2015-05-29 19:32 ` Sandra Loosemore 2015-05-30 4:50 ` Mike Frysinger 2015-05-30 16:09 ` Sandra Loosemore 2015-05-30 19:13 ` Mike Frysinger
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).