public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] ppc/svp64: test setvl ms operand
@ 2022-09-08  7:52 Dmitry Selyutin
  2022-09-08  7:54 ` Dmitry Selyutin
  2022-09-13 23:40 ` Alan Modra
  0 siblings, 2 replies; 7+ messages in thread
From: Dmitry Selyutin @ 2022-09-08  7:52 UTC (permalink / raw)
  To: binutils; +Cc: Alan Modra, Luke Kenneth Casson Leighton, Dmitry Selyutin

---
 gas/testsuite/gas/ppc/setvl.d | 1 +
 gas/testsuite/gas/ppc/setvl.s | 1 +
 2 files changed, 2 insertions(+)

diff --git a/gas/testsuite/gas/ppc/setvl.d b/gas/testsuite/gas/ppc/setvl.d
index a241c64fd16..4518ea32720 100644
--- a/gas/testsuite/gas/ppc/setvl.d
+++ b/gas/testsuite/gas/ppc/setvl.d
@@ -13,3 +13,4 @@ Disassembly of section \.text:
 .*:	(36 7e 00 58|58 00 7e 36) 	setvl   r0,r0,64,0,0,0
 .*:	(76 00 00 58|58 00 00 76) 	setvl   r0,r0,1,1,0,0
 .*:	(b6 00 00 58|58 00 00 b6) 	setvl   r0,r0,1,0,1,0
+.*:	(36 01 00 58|58 00 01 36) 	setvl   r0,r0,1,0,0,1
diff --git a/gas/testsuite/gas/ppc/setvl.s b/gas/testsuite/gas/ppc/setvl.s
index b7f1825b54b..a50820dc308 100644
--- a/gas/testsuite/gas/ppc/setvl.s
+++ b/gas/testsuite/gas/ppc/setvl.s
@@ -5,3 +5,4 @@ setvl 0,31,1,0,0,0
 setvl 0,0,64,0,0,0
 setvl 0,0,1,1,0,0
 setvl 0,0,1,0,1,0
+setvl 0,0,1,0,0,1
-- 
2.37.2


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ppc/svp64: test setvl ms operand
  2022-09-08  7:52 [PATCH] ppc/svp64: test setvl ms operand Dmitry Selyutin
@ 2022-09-08  7:54 ` Dmitry Selyutin
  2022-09-13 23:40 ` Alan Modra
  1 sibling, 0 replies; 7+ messages in thread
From: Dmitry Selyutin @ 2022-09-08  7:54 UTC (permalink / raw)
  To: binutils; +Cc: Alan Modra, Luke Kenneth Casson Leighton, Jan Beulich

On Thu, Sep 8, 2022 at 10:52 AM Dmitry Selyutin <ghostmansd@gmail.com> wrote:
>  gas/testsuite/gas/ppc/setvl.d | 1 +
>  gas/testsuite/gas/ppc/setvl.s | 1 +
>  2 files changed, 2 insertions(+)

Cc'ed Jan who actually found this issue.

-- 
Best regards,
Dmitry Selyutin

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ppc/svp64: test setvl ms operand
  2022-09-08  7:52 [PATCH] ppc/svp64: test setvl ms operand Dmitry Selyutin
  2022-09-08  7:54 ` Dmitry Selyutin
@ 2022-09-13 23:40 ` Alan Modra
  2022-09-15  8:27   ` Dmitry Selyutin
  1 sibling, 1 reply; 7+ messages in thread
From: Alan Modra @ 2022-09-13 23:40 UTC (permalink / raw)
  To: Dmitry Selyutin; +Cc: binutils, Luke Kenneth Casson Leighton

On Thu, Sep 08, 2022 at 10:52:15AM +0300, Dmitry Selyutin wrote:
> ---
>  gas/testsuite/gas/ppc/setvl.d | 1 +
>  gas/testsuite/gas/ppc/setvl.s | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/gas/testsuite/gas/ppc/setvl.d b/gas/testsuite/gas/ppc/setvl.d
> index a241c64fd16..4518ea32720 100644
> --- a/gas/testsuite/gas/ppc/setvl.d
> +++ b/gas/testsuite/gas/ppc/setvl.d
> @@ -13,3 +13,4 @@ Disassembly of section \.text:
>  .*:	(36 7e 00 58|58 00 7e 36) 	setvl   r0,r0,64,0,0,0
>  .*:	(76 00 00 58|58 00 00 76) 	setvl   r0,r0,1,1,0,0
>  .*:	(b6 00 00 58|58 00 00 b6) 	setvl   r0,r0,1,0,1,0
> +.*:	(36 01 00 58|58 00 01 36) 	setvl   r0,r0,1,0,0,1
> diff --git a/gas/testsuite/gas/ppc/setvl.s b/gas/testsuite/gas/ppc/setvl.s
> index b7f1825b54b..a50820dc308 100644
> --- a/gas/testsuite/gas/ppc/setvl.s
> +++ b/gas/testsuite/gas/ppc/setvl.s
> @@ -5,3 +5,4 @@ setvl 0,31,1,0,0,0
>  setvl 0,0,64,0,0,0
>  setvl 0,0,1,1,0,0
>  setvl 0,0,1,0,1,0
> +setvl 0,0,1,0,0,1
> -- 
> 2.37.2

OK.  Patches like this one can be applied as obvious, ie. not needing
PowerPC or global maintainer approval.

-- 
Alan Modra
Australia Development Lab, IBM

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ppc/svp64: test setvl ms operand
  2022-09-13 23:40 ` Alan Modra
@ 2022-09-15  8:27   ` Dmitry Selyutin
  2022-09-15  9:34     ` Alan Modra
  0 siblings, 1 reply; 7+ messages in thread
From: Dmitry Selyutin @ 2022-09-15  8:27 UTC (permalink / raw)
  To: Alan Modra; +Cc: binutils, Luke Kenneth Casson Leighton

On Wed, Sep 14, 2022 at 2:40 AM Alan Modra <amodra@gmail.com> wrote:
> OK.  Patches like this one can be applied as obvious, ie. not needing
> PowerPC or global maintainer approval.

Hi Alan, is there a guide on how to enable write access for the
repository? I cannot push this patch.


-- 
Best regards,
Dmitry Selyutin

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ppc/svp64: test setvl ms operand
  2022-09-15  8:27   ` Dmitry Selyutin
@ 2022-09-15  9:34     ` Alan Modra
  2022-09-20 21:33       ` Dmitry Selyutin
  0 siblings, 1 reply; 7+ messages in thread
From: Alan Modra @ 2022-09-15  9:34 UTC (permalink / raw)
  To: Dmitry Selyutin; +Cc: binutils, Luke Kenneth Casson Leighton

On Thu, Sep 15, 2022 at 11:27:43AM +0300, Dmitry Selyutin wrote:
> On Wed, Sep 14, 2022 at 2:40 AM Alan Modra <amodra@gmail.com> wrote:
> > OK.  Patches like this one can be applied as obvious, ie. not needing
> > PowerPC or global maintainer approval.
> 
> Hi Alan, is there a guide on how to enable write access for the
> repository? I cannot push this patch.

If you already have an account on sourceware.org, then send an email
to overseers@sourceware.org asking for write access to binutils-gdb,
nominating me as sponsor.  If you don't have an account, follow the
directions at https://sourceware.org/

-- 
Alan Modra
Australia Development Lab, IBM

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ppc/svp64: test setvl ms operand
  2022-09-15  9:34     ` Alan Modra
@ 2022-09-20 21:33       ` Dmitry Selyutin
  2022-09-20 21:45         ` Dmitry Selyutin
  0 siblings, 1 reply; 7+ messages in thread
From: Dmitry Selyutin @ 2022-09-20 21:33 UTC (permalink / raw)
  To: Alan Modra; +Cc: binutils, Luke Kenneth Casson Leighton

Hi Alain, sorry it took that long. I've just pushed that trivial
patch, please let me know if there are any issues.

As for the larger patches for SVP64 support, I'm entering yet another
round of bureaucratic procedures, and waiting for papers to be signed
by my new employer. However, this is OK, since I'm anyway refactoring
the code so that it relies on machine-readable specs we have in
libresoc, and does not depend on an ad-hoc solution implemented
initially but not submitted yet. So it might take a while before I
start submitting the patches to support the actual prefixed
instructions.

-- 
Best regards,
 Dmitry Selyutin

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ppc/svp64: test setvl ms operand
  2022-09-20 21:33       ` Dmitry Selyutin
@ 2022-09-20 21:45         ` Dmitry Selyutin
  0 siblings, 0 replies; 7+ messages in thread
From: Dmitry Selyutin @ 2022-09-20 21:45 UTC (permalink / raw)
  To: Alan Modra; +Cc: binutils, Luke Kenneth Casson Leighton

On Wed, Sep 21, 2022 at 12:33 AM Dmitry Selyutin <ghostmansd@gmail.com> wrote:
> Hi Alain, sorry it took that long. I've just pushed that trivial
> patch, please let me know if there are any issues.

Oops, incidentally created a branch called upstream. Should be in
master now. Should I delete the upstream branch I created?

-- 
Best regards,
Dmitry Selyutin

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2022-09-20 21:46 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-08  7:52 [PATCH] ppc/svp64: test setvl ms operand Dmitry Selyutin
2022-09-08  7:54 ` Dmitry Selyutin
2022-09-13 23:40 ` Alan Modra
2022-09-15  8:27   ` Dmitry Selyutin
2022-09-15  9:34     ` Alan Modra
2022-09-20 21:33       ` Dmitry Selyutin
2022-09-20 21:45         ` Dmitry Selyutin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).