This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFA: Fix PR middle-end/29847


Joern RENNECKE <joern.rennecke@st.com> writes:
> Both to avoid problems with existing code relying on the current
> behaviour, and to accomodate cases where endianness varies between
> register classes (and/or is neither big nor little endian), I have
> introduced a new target hook TARGET_MATCH_ADJUST, which defaults to
> the current behaviour.
>
> I've regression tested the code patches on i686-pc-linux-gnu with a
> native bootstrap, and also as part of the patches for PR29845 on
> i686-pc-linux-gnu X sh-elf.  I've tested the tm.texi patch with 'make
> info' and inspecting gccint.info with vi and info.

FWIW, I've looked at the patch for 29845 too, but could you explain
in more detail why you think this is the right fix?  I would have
thought that matching constraints should always provide the least
significant end of the wider operand, and if gcc doesn't know which
end that is in a particular situation, it should be taught in a more
direct way.  Introducing a target macro at the matching level seems
a bit hackish.

Richard


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]