On Sat, Nov 5, 2022 at 12:25 PM Richard Biener
wrote:
>
> On Wed, Nov 2, 2022 at 1:46 PM Uros Bizjak wrote:
> >
> > On Wed, Nov 2, 2022 at 1:45 PM Robin Dapp wrote:
> > >
> > > > IIRC, I was trying to "fix" modeless operand by giving it a mode, but
> > > > since it made no difference for x86, I
On Wed, Nov 2, 2022 at 1:46 PM Uros Bizjak wrote:
>
> On Wed, Nov 2, 2022 at 1:45 PM Robin Dapp wrote:
> >
> > > IIRC, I was trying to "fix" modeless operand by giving it a mode, but
> > > since it made no difference for x86, I later dropped the patch.
> > > However, operand with a known mode is
On Wed, Nov 2, 2022 at 1:45 PM Robin Dapp wrote:
>
> > IIRC, I was trying to "fix" modeless operand by giving it a mode, but
> > since it made no difference for x86, I later dropped the patch.
> > However, operand with a known mode is preferred, so if it works for
> > you, just include my patch
> IIRC, I was trying to "fix" modeless operand by giving it a mode, but
> since it made no difference for x86, I later dropped the patch.
> However, operand with a known mode is preferred, so if it works for
> you, just include my patch in your submission. My patch is somehow
> trivial if we want
On Wed, Nov 2, 2022 at 1:12 PM Robin Dapp wrote:
>
> Hi,
>
> > With the patch my local changes to make better use of vec_set work
> > nicely even though I haven't done a full bootstrap yet. Were there
> > other issues with the patch or can it still be applied?
>
> I performed a bootstrap as well
Hi,
> With the patch my local changes to make better use of vec_set work
> nicely even though I haven't done a full bootstrap yet. Were there
> other issues with the patch or can it still be applied?
I performed a bootstrap as well as a regtest with -march=z16 on s390.
There is no new fallout.
Hi,
I'm looking into vec_set with variable index on s390. Uros posted a
patch [1] that did not make it upstream in Nov 2020. It changed the
mode of the index operand to whatever the target supports in
can_vec_set_var_idx_p. I missed it back then but we indeed do not make
proper use of vec_set
