This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: AW: Wonly-top-basic-asm
- From: David Wohlferd <dw at LimeGreenSocks dot com>
- To: Bernd Edlinger <bernd dot edlinger at hotmail dot de>, Bernd Schmidt <bschmidt at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Richard Henderson <rth at redhat dot com>, "jason at redhat dot com" <jason at redhat dot com>
- Cc: "segher at kernel dot crashing dot org" <segher at kernel dot crashing dot org>, "sandra at codesourcery dot com" <sandra at codesourcery dot com>, "Paul_Koning at Dell dot com" <Paul_Koning at Dell dot com>, Jeff Law <law at redhat dot com>, "bernds_cb1 at t-online dot de" <bernds_cb1 at t-online dot de>, Andrew Haley <aph at redhat dot com>
- Date: Mon, 8 Feb 2016 12:14:59 -0800
- Subject: Re: AW: Wonly-top-basic-asm
- Authentication-results: sourceware.org; auth=none
- References: <56A54EF9 dot 8060006 at LimeGreenSocks dot com> <56A61442 dot 3090803 at redhat dot com> <56A9C134 dot 1030500 at LimeGreenSocks dot com> <56B80F57 dot 9020606 at LimeGreenSocks dot com> <HE1PR07MB09059C5C9C060394C61A1232E4D50 at HE1PR07MB0905 dot eurprd07 dot prod dot outlook dot com>
On 2/7/2016 10:45 PM, Bernd Edlinger wrote:
On 8. 2. 2016 04:45, David Wohlferd wrote:
I replied with a patch that includes most of the changes you asked for
(see inline below). Were you waiting on me for something more?
ChangeLog entries are still missing.
I'll add them back in the next post.
David, there is a tool that you can use to check the patch
for some style-nits before submission.
I was not aware of this tool. I'll fix these before the next post.
At one point you proposed renaming the option -Wbasic-asm. This seemed
a little terse (and possibly misleading) so I counter-proposed
-Wbasic-asm-in-function (a little verbose, but clearer). I have no
strong preferences here, and you haven't said one way or the other. Are
we just going to stick with -Wonly-top-basic-asm?
Hopefully one more try and this will be done.
Thanks,
dw