On Mon, 22 Feb 2016, Nathan Sidwell wrote:
> On 02/15/16 13:44, Alexander Monakov wrote:
> > This patch implements the NVPTX backend part of the transition to
>
> > + static const char template64[] = ENTRY_TEMPLATE ("64", "8",
> > "mad.wide.u32");
> > + static const char template32[] = ENTRY_TEMPLATE ("32", "4", "mad.lo.u32
> > ");
> > +#undef ENTRY_TEMPLATE
> > + const char *template_1 = TARGET_ABI64 ? template64 : template32;
> > + const char *template_2 = template_1 + strlen (template64) + 1;
> ^^^
> this looks mighty suspicious -- are you presuming some specific placement of
> template64 & template32? (and even then I think it'll only work for
> TARGET_ABI64)
Template strings have an embedded nul character at the position where ORIG
goes, so template_2 is set to point at the position following the embedded nul
in template_1. Offset of the embedded nul is the same in each template
string, so it doesn't matter which goes into the argument of strlen (but
supplying template64 or template32 instead of template_1 allows easier folding).
Alexander
