[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 10/11] tests/tcg/s390x: Tests for Vector Enhancements Faci
From: |
David Miller |
Subject: |
Re: [PATCH v4 10/11] tests/tcg/s390x: Tests for Vector Enhancements Facility 2 |
Date: |
Tue, 5 Apr 2022 13:03:32 -0400 |
Recommendation for comment?
/* vri-d encoding matches vrr for 4b imm.
.insn does not handle this encoding variant.
*/
Christian: I will push another patch version as soon as that's decided.
(unless you prefer to choose the comment and edit during staging)
On Tue, Apr 5, 2022 at 6:13 AM David Hildenbrand <david@redhat.com> wrote:
>
> On 01.04.22 17:25, Christian Borntraeger wrote:
> > Am 01.04.22 um 17:02 schrieb David Miller:
> >> vrr is almost a perfect match (it is for this, larger than imm4 would
> >> need to be split).
> >>
> >> .long : this would be uglier.
> >> use enough to be filled with nops after ?
> >> or use a 32b and 16b instead if it's in .text it should make no difference.
> >
> > I will let Richard or David decide what they prefer.
> >
>
> I don't particularly care as long as there is a comment stating why we
> need this hack.
>
> --
> Thanks,
>
> David / dhildenb
>