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.
Jakub
