Re: GAS patch for sh*-unknown-linux-gnu

[NIIBE Yutaka <gniibe at m17n dot org>]

Ralf Corsepius wrote:
 > Well, I am not sure if this patch is a good idea.
 > IMHO, it introduces compatibility problems and complications at doubtful
 > benefits:

Could you please read the patch, and possibly test it?
It doesn't change any sh target except sh*-unknown-linux-gnu (except
redundant -big).

 > * backward compatiblity problems. Users are used to sh-binutils to
 > default to big endianness, all gcc version up to now have pre-supposed
 > using bigendian as default and packages might be relying on it.

Yes, I know.  I did care, and I believe that I maintain backward
compatibility.

If you insist it's not, could you please let me know where's backward
compatibility problem, how?

 > * -big would be redundant to -little and actually is of little use for
 > all sh users but linux-gnu users.

When little endian is default, we need -big.

 > Additionally, I also do not understand why a new variable
 > "target_big_endian" shall be used instead of "shl". AFAIU, it breaks
 > sourcecode backward compatibility at no real functional benefits.

It is not new variable.  The variable "target_big_endian" is used
every other targets.  SH target uses "shl" in some not clear reason.

Could you please look other use of "target_big_endian" in other targets.
--