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