Re: [patches] Re: [PATCH] RISC-V: Pass --no-relax to linker if -mno-relax is present.

[Jim Wilson <jimw at sifive dot com>]

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