This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: [PATCH] Fix ia64 gas dependency warnings
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jim Wilson <wilson at specifixinc dot com>
- Cc: binutils at sources dot redhat dot com
- Date: Tue, 8 Jun 2004 20:31:03 +0200
- Subject: Re: [PATCH] Fix ia64 gas dependency warnings
- References: <20040531221153.GM4736@devserv.devel.redhat.com> <1086052053.1190.5.camel@leaf.tuliptree.org>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, May 31, 2004 at 06:07:32PM -0700, Jim Wilson wrote:
> > * ia64-gen.c (in_iclass): Handle more postinc st
> > and ld variants.
>
> This looks right to me. Testcases for the testsuite would be a good
> idea.
Thanks, this is the test that I've commited together with the patch:
2004-06-08 Jakub Jelinek <jakub@redhat.com>
* gas/ia64/dv-raw-err.s: Add some new postinc tests.
* gas/ia64/dv-raw-err.l: Updated.
--- gas/testsuite/gas/ia64/dv-raw-err.s.jj 2000-09-22 21:43:48.000000000 +0200
+++ gas/testsuite/gas/ia64/dv-raw-err.s 2004-06-08 22:15:00.355498976 +0200
@@ -576,4 +576,21 @@
(p27) br.cond.sptk b1 // no DV here
;;
-L:
+// postinc
+ st8 [r6] = r8, 16
+ add r7 = 8, r6 // impliedf
+ ;;
+ ldfd f14 = [r6], 16
+ add r7 = 8, r6 // impliedf
+ ;;
+ stfd [r6] = f14, 16
+ add r7 = r8, r6
+ ;;
+ add r6 = 8, r7
+ ld8 r8 = [r6], 16 // impliedf, WAW
+ ;;
+ add r6 = 8, r7
+ ldfd f14 = [r6], 16 // impliedf, WAW
+ ;;
+
+L:
--- gas/testsuite/gas/ia64/dv-raw-err.l.jj 2000-09-22 21:43:48.000000000 +0200
+++ gas/testsuite/gas/ia64/dv-raw-err.l 2004-06-08 22:16:30.145407455 +0200
@@ -277,3 +277,17 @@
.*:569: Warning: This is the location of the conflicting usage
.*:573: Warning: Use of 'br.cond.sptk' may violate RAW dependency 'PR%, % in 16 - 62' \(impliedf\), specific resource number is 25
.*:572: Warning: This is the location of the conflicting usage
+.*:581: Warning: Use of 'adds' .* RAW dependency 'GR%, % in 1 - 127' \(impliedf\), specific resource number is 6
+.*:580: Warning: This is the location of the conflicting usage
+.*:584: Warning: Use of 'adds' .* RAW dependency 'GR%, % in 1 - 127' \(impliedf\), specific resource number is 6
+.*:583: Warning: This is the location of the conflicting usage
+.*:587: Warning: Use of 'add' .* RAW dependency 'GR%, % in 1 - 127' \(impliedf\), specific resource number is 6
+.*:586: Warning: This is the location of the conflicting usage
+.*:590: Warning: Use of 'ld8' .* RAW dependency 'GR%, % in 1 - 127' \(impliedf\), specific resource number is 6
+.*:589: Warning: This is the location of the conflicting usage
+.*:590: Warning: Use of 'ld8' .* WAW dependency 'GR%, % in 1 - 127' \(impliedf\), specific resource number is 6
+.*:589: Warning: This is the location of the conflicting usage
+.*:593: Warning: Use of 'ldfd' .* RAW dependency 'GR%, % in 1 - 127' \(impliedf\), specific resource number is 6
+.*:592: Warning: This is the location of the conflicting usage
+.*:593: Warning: Use of 'ldfd' .* WAW dependency 'GR%, % in 1 - 127' \(impliedf\), specific resource number is 6
+.*:592: Warning: This is the location of the conflicting usage
Jakub