https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93582

--- Comment #23 from rguenther at suse dot de <rguenther at suse dot de> ---
On Tue, 11 Feb 2020, jakub at gcc dot gnu.org wrote:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93582
> 
> --- Comment #22 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> (In reply to Richard Biener from comment #21)
> > The shift_bytes_in_array_{left,right} routines should go next to
> > native_{encode,interpret} where maybe also a comment should indicate how to
> > combine both?
> 
> Ok.
> 
> > The vn_reference_lookup_3 part looks OK to me, the visit_nary_op one is a 
> > bit
> > gross ... some comments:
> 
> Yes, that part was meant as a hack that we could hopefully throw away for
> GCC11.
> Anyway, I'll try to finish up the vn_reference_lookup_3 part for big endian,
> add testcases and submit that, then perhaps try to do also the pieces stuff 
> for
> non-byte granularity incrementally, and leave the rest for last.

Thanks.  Guess there's some helpers to factor out - memset() def
handling also currently requires byte-offsets (if it's not a clearing
memset).  It elides the native_encode part but still uses
native_interpret for the rest.

Reply via email to