> -----Original Message-----
> From: Jakub Jelinek <ja...@redhat.com>
> Sent: Wednesday, August 7, 2024 11:27 PM
> To: Prathamesh Kulkarni <prathame...@nvidia.com>
> Cc: Richard Biener <rguent...@suse.de>; Richard Sandiford
> <richard.sandif...@arm.com>; gcc-patches@gcc.gnu.org
> Subject: Re: Support streaming of poly_int for offloading when it's
> degree <= accel's NUM_POLY_INT_COEFFS
>
> External email: Use caution opening links or attachments
>
>
> On Wed, Aug 07, 2024 at 05:53:00PM +0000, Prathamesh Kulkarni wrote:
> > > Two spaces after . rather than just one, and because of that move
> in
> > > on the next line.
> > >
> > > > + host_num_poly_int_coeffs. */
> > >
> > > Otherwise LGTM.
> > Thanks, I have adjusted the formatting of the comment and a typo in
> streamer_read_poly_uint64.
> > Patch passes bootstrap+test and LTO bootstrap+test on aarch64-linux-
> gnu, LTO bootstrap+test on x86_64-linux-gnu.
> > And doesn't seem to regress libgomp testing for x86_64 -> nvptx
> offloading (altho there were a few occurrences of flaky tests in
> results).
> > Is the patch OK to commit ?
>
> The "Otherwise LGTM." was already covering that.
> So yes.
Thanks, pushed to trunk in:
https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=38900247f3880d6eca2e364a000e5898f8deae64
Thanks,
Prathamesh
>
> Jakub
