> -Original Message-
> From: Jakub Jelinek
> Sent: Wednesday, August 7, 2024 11:27 PM
> To: Prathamesh Kulkarni
> Cc: Richard Biener ; Richard Sandiford
> ; gcc-patches@gcc.gnu.org
> Subject: Re: Support streaming of poly_int for offloading when it&
On Wed, Aug 07, 2024 at 05:53:00PM +, 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 typ
> -Original Message-
> From: Jakub Jelinek
> Sent: Monday, August 5, 2024 8:01 PM
> To: Prathamesh Kulkarni
> Cc: Richard Biener ; Richard Sandiford
> ; gcc-patches@gcc.gnu.org
> Subject: Re: Support streaming of poly_int for offloading when it's
> degree &
On Mon, Aug 05, 2024 at 02:24:00PM +, Prathamesh Kulkarni wrote:
> gcc/ChangeLog:
> PR ipa/96265
> PR ipa/111937
> * data-streamer-in.cc (streamer_read_poly_uint64): Remove code for
> streaming, and call poly_int_read_common instead.
> (streamer_read_poly_int64):
> -Original Message-
> From: Jakub Jelinek
> Sent: Friday, August 2, 2024 5:43 PM
> To: Prathamesh Kulkarni
> Cc: Richard Biener ; Richard Sandiford
> ; gcc-patches@gcc.gnu.org
> Subject: Re: Support streaming of poly_int for offloading when it's
> degree &
On Fri, Aug 02, 2024 at 11:58:19AM +, Prathamesh Kulkarni wrote:
> diff --git a/gcc/data-streamer-in.cc b/gcc/data-streamer-in.cc
> index 7dce2928ef0..7b9d8cc0129 100644
> --- a/gcc/data-streamer-in.cc
> +++ b/gcc/data-streamer-in.cc
> @@ -182,10 +182,8 @@ streamer_read_hwi (class lto_input_blo
> -Original Message-
> From: Jakub Jelinek
> Sent: Wednesday, July 31, 2024 8:46 PM
> To: Prathamesh Kulkarni
> Cc: Richard Biener ; Richard Sandiford
> ; gcc-patches@gcc.gnu.org
> Subject: Re: Support streaming of poly_int for offloading when it&
On Wed, Jul 31, 2024 at 02:58:34PM +, Prathamesh Kulkarni wrote:
> There are a couple of issues in the patch:
> (1) The patch streams out NUM_POLY_INT_COEFFS at beginning of mode_table,
> which should work for bp_unpack_poly_value,
> (since AFAIK, it's only called by lto_input_mode_table). How
> -Original Message-
> From: Tobias Burnus
> Sent: Tuesday, July 30, 2024 6:08 PM
> To: Prathamesh Kulkarni ; gcc-
> patc...@gcc.gnu.org
> Subject: Re: Support streaming of poly_int for offloading when it's
> degree <= accel's NUM_POLY_INT_COEFFS
>
> -Original Message-
> From: Prathamesh Kulkarni
> Sent: Tuesday, July 30, 2024 4:44 PM
> To: Jakub Jelinek ; Richard Biener
>
> Cc: Richard Sandiford ; gcc-
> patc...@gcc.gnu.org
> Subject: RE: Support streaming of poly_int for offloading when
Prathamesh Kulkarni wrote:
Thanks for your suggestions on RFC email, the attached patch adds support for
streaming of poly_int when it's degree <= accel's NUM_POLY_INT_COEFFS.
First, thanks a lot for your patch!
Secondly, it seems as if this patch is indented to fully or partially
fix the fo
> -Original Message-
> From: Jakub Jelinek
> Sent: Tuesday, July 30, 2024 3:16 PM
> To: Richard Biener
> Cc: Richard Sandiford ; Prathamesh Kulkarni
> ; gcc-patches@gcc.gnu.org
> Subject: Re: Support streaming of poly_int for offloading when it&
Jakub Jelinek writes:
> On Tue, Jul 30, 2024 at 11:25:42AM +0200, Richard Biener wrote:
>> Only "relevant" stuff should be streamed - the offload code and all
>> trees refered to.
>
> Yeah.
>
>> > > I think all current issues are because of poly-* leaking in for cases
>> > > where a non-poly would
On Tue, Jul 30, 2024 at 11:25:42AM +0200, Richard Biener wrote:
> Only "relevant" stuff should be streamed - the offload code and all
> trees refered to.
Yeah.
> > > I think all current issues are because of poly-* leaking in for cases
> > > where a non-poly would have worked fine, but I have not
gt;> >>
> >> >> > -Original Message-
> >> >> > From: Richard Sandiford
> >> >> > Sent: Monday, July 29, 2024 9:43 PM
> >> >> > To: Richard Biener
> >> >> > Cc: Prathamesh Kul
> From: Richard Sandiford
>> >> > Sent: Monday, July 29, 2024 9:43 PM
>> >> > To: Richard Biener
>> >> > Cc: Prathamesh Kulkarni ; gcc-
>> >> > patc...@gcc.gnu.org
>> >> > Subject: Re: Support streami
uly 29, 2024 9:43 PM
> >> > To: Richard Biener
> >> > Cc: Prathamesh Kulkarni ; gcc-
> >> > patc...@gcc.gnu.org
> >> > Subject: Re: Support streaming of poly_int for offloading when it's
> >> > degree <= accel's NUM_POLY_INT
karni ; gcc-
>> > patc...@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 Tue, 30 Jul 2024, Prathamesh Kulkarni wrote:
>
>
> > -Original Message-
> > From: Richard Sandiford
> > Sent: Monday, July 29, 2024 9:43 PM
> > To: Richard Biener
> > Cc: Prathamesh Kulkarni ; gcc-
> > patc...@gcc.gnu.org
> >
> -Original Message-
> From: Richard Sandiford
> Sent: Monday, July 29, 2024 9:43 PM
> To: Richard Biener
> Cc: Prathamesh Kulkarni ; gcc-
> patc...@gcc.gnu.org
> Subject: Re: Support streaming of poly_int for offloading when it's
> degree <= accel
Richard Biener writes:
> On Mon, 29 Jul 2024, Prathamesh Kulkarni wrote:
>
>> Hi Richard,
>> Thanks for your suggestions on RFC email, the attached patch adds support
>> for streaming of poly_int when it's degree <= accel's NUM_POLY_INT_COEFFS.
>> The patch changes streaming of poly_int as follow
On Mon, 29 Jul 2024, Prathamesh Kulkarni wrote:
> Hi Richard,
> Thanks for your suggestions on RFC email, the attached patch adds support for
> streaming of poly_int when it's degree <= accel's NUM_POLY_INT_COEFFS.
> The patch changes streaming of poly_int as follows:
>
> Streaming out poly_int:
22 matches
Mail list logo