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

Reply via email to