Messages in this thread | | | Date | Thu, 16 Mar 2017 14:04:01 -0500 | From | Josh Poimboeuf <> | Subject | Re: [PATCH] x86: mostly disable '-maccumulate-outgoing-args' |
| |
On Thu, Mar 16, 2017 at 01:53:05PM -0500, Josh Poimboeuf wrote: > On Thu, Mar 16, 2017 at 01:36:35PM -0500, Josh Poimboeuf wrote: > > On Thu, Mar 16, 2017 at 01:32:01PM -0400, Steven Rostedt wrote: > > > On Thu, 16 Mar 2017 10:42:08 -0500 > > > Josh Poimboeuf <jpoimboe@redhat.com> wrote: > > > > + ACCUMULATE_OUTGOING_ARGS := 1 > > > > + endif > > > > + endif > > > > +endif > > > > + > > > > +# Jump labels need '-maccumulate-outgoing-args' for gcc < 4.5.2 to prevent > > > > > > Can we make a test instead? I hate testing versions, and things get > > > backported all the time. We usually like to have a test case instead of > > > relying on versions. Not to mention, a newer gcc may one day break. > > > > Tests are generally better, but I'm not sure how to test for this > > cleanly. The test is rather big for embedding in a makefile: > > > > https://gcc.gnu.org/bugzilla/attachment.cgi?id=22199 > > > > Any ideas? > > After some snooping I discovered there's some precedent for doing this > in the scripts/gcc-*.sh files. So maybe I'll add a test there and call > it from the Makefile.
But now I realize that those other tests are just build tests, whereas this one needs to be executed. That's a no-go for cross compilers. So I think we need to do the version check after all.
-- Josh
| |