On 04/18/2018 06:59 PM, Kito Cheng wrote:
Hi Jim:
Turned off both the assembler and the linker sounds good idea to me,
but it's not support on current assembler now, and gcc might release in
next few month, so I afraid we'll have a short time gap that is
-mno-relax is broken due to assembler not support that command
line option in the latest release yet?
Sorry, I didn't look at this closely enough. We already have a
-mrelax/-mno-relax added by Palmer. You even mentioned that. So all
your patch is doing is adding support to pass it to the linker, which is OK.
You are right that we can't release a compiler that uses the new
assembler option until after there is an assembler release, so we can't
modify gcc to use it until after the gcc-8 release branch is made. But
we don't need it for now, as we have Palmer's patch. I still think it
was a good idea to add the assembler option though, to make it a little
more user friendly.
Anyways, this patch is OK. I have committed it.
Jim
