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: [committed] PR target/25871: Fix MIPS trampolines for Pmode == DImode


Thiemo Seufer <ths@networkno.de> writes:
> Richard Sandiford wrote:
>> +#define TRAMPOLINE_TEMPLATE(STREAM)					\
>> +{									\
>> +  if (ptr_mode == DImode)						\
>> +    fprintf (STREAM, "\t.word\t0x03e0082d\t\t# dmove   $1,$31\n");	\
>> +  else									\
>> +    fprintf (STREAM, "\t.word\t0x03e00821\t\t# move   $1,$31\n");	\
>
> It probably makes sense to use explicitly "addu" and "daddu" in the
> comments, this avoids confusion caused by a non-existing "dmove"
> assembler macro.

I disagree.  It's a comment after all, and I think the meaning
is crystal clear.

Richard


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