public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] add bdver2 processor.
@ 2011-05-04 16:16 Quentin Neill
  2011-05-04 16:45 ` H.J. Lu
  0 siblings, 1 reply; 12+ messages in thread
From: Quentin Neill @ 2011-05-04 16:16 UTC (permalink / raw)
  To: binutils

[-- Attachment #1: Type: text/plain, Size: 595 bytes --]

This patch adds the bdver2 processor flag.  It passes make check on x86-64.

Okay to commit?
-- 
Quentin


gas/
        * config/tc-i386.c (cpu_arch): Add bdver2.
        (i386_align_code): Add PROCESSOR_BDVER2 cases.
        * config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
        * doc/c-i386.texi: Add bdver2.

opcodes/
        * i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
        * i386-init.h: Regenerated.

gas/testsuite/
        * gas/i386/i386.exp: Add new bdver2 test cases.
        * gas/i386/nops-1-bdver2.d: New.
        * gas/i386/x86-64-nops-1-bdver2.d: New.

[-- Attachment #2: 0001-2011-05-04-Quentin-Neill-quentin.neill-amd.com.patch --]
[-- Type: text/x-patch, Size: 20658 bytes --]

From 84066b9f4ed02a2fb2560f20ad586e9d258b806d Mon Sep 17 00:00:00 2001
From: Quentin Neill <quentin.neill@amd.com>
Date: Wed, 4 May 2011 09:39:03 -0500
Subject: [PATCH] 2011-05-04  Quentin Neill  <quentin.neill@amd.com>

gas/
	* config/tc-i386.c (cpu_arch): Add bdver2.
	(i386_align_code): Add PROCESSOR_BDVER2 cases.
	* config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
	* doc/c-i386.texi: Add bdver2.

opcodes/
	* i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
	* i386-init.h: Regenerated.

gas/testsuite/
	* gas/i386/i386.exp: Add new bdver2 test cases.
	* gas/i386/nops-1-bdver2.d: New.
	* gas/i386/x86-64-nops-1-bdver2.d: New.
---
 gas/ChangeLog                                 |    7 +
 gas/config/tc-i386.c                          |    8 +-
 gas/config/tc-i386.h                          |    3 +-
 gas/doc/c-i386.texi                           |    3 +-
 gas/testsuite/ChangeLog                       |    6 +
 gas/testsuite/gas/i386/i386.exp               |    2 +
 gas/testsuite/gas/i386/nops-1-bdver2.d        |  162 +++++++++++++++++++++++++
 gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d |  162 +++++++++++++++++++++++++
 opcodes/ChangeLog                             |    5 +
 opcodes/i386-gen.c                            |    2 +
 opcodes/i386-init.h                           |    5 +
 11 files changed, 361 insertions(+), 4 deletions(-)
 create mode 100644 gas/testsuite/gas/i386/nops-1-bdver2.d
 create mode 100644 gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 37ef434..83347d2 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,10 @@
+2011-05-04  Quentin Neill  <quentin.neill@amd.com>
+
+	* config/tc-i386.c (cpu_arch): Add bdver2.
+	(i386_align_code): Add PROCESSOR_BDVER2 cases.
+	* config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
+	* doc/c-i386.texi: Add bdver2.
+
 2011-05-03  Paul Brook  <paul@codesourcery.com>
 
 
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index 69ac150..5f24306 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -647,6 +647,8 @@ static const arch_entry cpu_arch[] =
     CPU_AMDFAM10_FLAGS, 0, 0 },
   { STRING_COMMA_LEN ("bdver1"), PROCESSOR_BDVER1,
     CPU_BDVER1_FLAGS, 0, 0 },
+  { STRING_COMMA_LEN ("bdver2"), PROCESSOR_BDVER2,
+    CPU_BDVER2_FLAGS, 0, 0 },
   { STRING_COMMA_LEN (".8087"), PROCESSOR_UNKNOWN,
     CPU_8087_FLAGS, 0, 0 },
   { STRING_COMMA_LEN (".287"), PROCESSOR_UNKNOWN,
@@ -1023,8 +1025,8 @@ i386_align_code (fragS *fragP, int count)
      PROCESSOR_CORE, PROCESSOR_CORE2, PROCESSOR_COREI7, and
      PROCESSOR_GENERIC64, alt_long_patt will be used.
      3. For PROCESSOR_ATHLON, PROCESSOR_K6, PROCESSOR_K8 and
-     PROCESSOR_AMDFAM10, and PROCESSOR_BDVER1, alt_short_patt
-     will be used.
+     PROCESSOR_AMDFAM10, PROCESSOR_BDVER1, and PROCESSOR_BDVER2
+     alt_short_patt will be used.
 
      When -mtune= isn't used, alt_long_patt will be used if
      cpu_arch_isa_flags has CpuNop.  Otherwise, f32_patt will
@@ -1077,6 +1079,7 @@ i386_align_code (fragS *fragP, int count)
 	    case PROCESSOR_K8:
 	    case PROCESSOR_AMDFAM10:
 	    case PROCESSOR_BDVER1:
+	    case PROCESSOR_BDVER2:
 	      patt = alt_short_patt;
 	      break;
 	    case PROCESSOR_I386:
@@ -1106,6 +1109,7 @@ i386_align_code (fragS *fragP, int count)
 	    case PROCESSOR_K8:
 	    case PROCESSOR_AMDFAM10:
 	    case PROCESSOR_BDVER1:
+	    case PROCESSOR_BDVER2:
 	    case PROCESSOR_GENERIC32:
 	      /* We use cpu_arch_isa_flags to check if we CAN optimize
 		 with nops.  */
diff --git a/gas/config/tc-i386.h b/gas/config/tc-i386.h
index bf227c6..4026999 100644
--- a/gas/config/tc-i386.h
+++ b/gas/config/tc-i386.h
@@ -230,7 +230,8 @@ enum processor_type
   PROCESSOR_GENERIC32,
   PROCESSOR_GENERIC64,
   PROCESSOR_AMDFAM10,
-  PROCESSOR_BDVER1
+  PROCESSOR_BDVER1,
+  PROCESSOR_BDVER2
 };
 
 extern enum processor_type cpu_arch_tune;
diff --git a/gas/doc/c-i386.texi b/gas/doc/c-i386.texi
index 34a69b0..bedff79 100644
--- a/gas/doc/c-i386.texi
+++ b/gas/doc/c-i386.texi
@@ -118,6 +118,7 @@ processor names are recognized:
 @code{k8},
 @code{amdfam10},
 @code{bdver1},
+@code{bdver2},
 @code{generic32} and
 @code{generic64}.
 
@@ -1000,7 +1001,7 @@ supported on the CPU specified.  The choices for @var{cpu_type} are:
 @item @samp{prescott} @tab @samp{nocona} @tab @samp{core} @tab @samp{core2}
 @item @samp{corei7} @tab @samp{l1om}
 @item @samp{k6} @tab @samp{k6_2} @tab @samp{athlon} @tab @samp{k8}
-@item @samp{amdfam10} @tab @samp{bdver1}
+@item @samp{amdfam10} @tab @samp{bdver1} @tab @samp{bdver2}
 @item @samp{generic32} @tab @samp{generic64}
 @item @samp{.mmx} @tab @samp{.sse} @tab @samp{.sse2} @tab @samp{.sse3}
 @item @samp{.ssse3} @tab @samp{.sse4.1} @tab @samp{.sse4.2} @tab @samp{.sse4}
diff --git a/gas/testsuite/ChangeLog b/gas/testsuite/ChangeLog
index 25618f8..bee7dad 100644
--- a/gas/testsuite/ChangeLog
+++ b/gas/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2010-05-04  Quentin Neill  <quentin.neill@amd.com>
+
+	* gas/i386/i386.exp: Add new bdver2 test cases.
+	* gas/i386/nops-1-bdver2.d: New.
+	* gas/i386/x86-64-nops-1-bdver2.d: New.
+
 2011-04-29  Hans-Peter Nilsson  <hp@axis.com>
 
 	* gas/elf/dwarf2-1.d, gas/elf/dwarf2-2.d: Adjust for change in
diff --git a/gas/testsuite/gas/i386/i386.exp b/gas/testsuite/gas/i386/i386.exp
index 0eb151f..cc732f8 100644
--- a/gas/testsuite/gas/i386/i386.exp
+++ b/gas/testsuite/gas/i386/i386.exp
@@ -82,6 +82,7 @@ if [expr ([istarget "i*86-*-*"] ||  [istarget "x86_64-*-*"]) && [gas_32_check]]
     run_dump_test "nops-1-k8"
     run_dump_test "nops-1-core2"
     run_dump_test "nops-1-bdver1"
+    run_dump_test "nops-1-bdver2"
     run_dump_test "nops-2"
     run_dump_test "nops-2-i386"
     run_dump_test "nops-2-core2"
@@ -316,6 +317,7 @@ if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_64_check]] t
     run_dump_test "x86-64-nops-1-core2"
     run_dump_test "x86-64-nops-1-pentium"
     run_dump_test "x86-64-nops-1-bdver1"
+    run_dump_test "x86-64-nops-1-bdver2"
     run_dump_test "x86-64-nops-2"
     run_dump_test "x86-64-nops-3"
     run_dump_test "x86-64-nops-4"
diff --git a/gas/testsuite/gas/i386/nops-1-bdver2.d b/gas/testsuite/gas/i386/nops-1-bdver2.d
new file mode 100644
index 0000000..d33d6b7
--- /dev/null
+++ b/gas/testsuite/gas/i386/nops-1-bdver2.d
@@ -0,0 +1,162 @@
+#as: -mtune=bdver2
+#source: nops-1.s
+#objdump: -drw
+#name: i386 -mtune=bdver2 nops 1
+
+.*: +file format .*
+
+
+Disassembly of section .text:
+
+0+ <nop15>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+[ 	]*[a-f0-9]+:	0f 1f 84 00 00 00 00 00 	nopl   0x0\(%eax,%eax,1\)
+
+0+10 <nop14>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+
+0+20 <nop13>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+
+0+30 <nop12>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+
+0+40 <nop11>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 44 00 00       	nopl   0x0\(%eax,%eax,1\)
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+
+0+50 <nop10>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 2e 0f 1f 84 00 00 00 00 00 	nopw   %cs:0x0\(%eax,%eax,1\)
+
+0+60 <nop9>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 84 00 00 00 00 00 	nopw   0x0\(%eax,%eax,1\)
+
+0+70 <nop8>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 84 00 00 00 00 00 	nopl   0x0\(%eax,%eax,1\)
+
+0+80 <nop7>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+
+0+90 <nop6>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+
+0+a0 <nop5>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 44 00 00       	nopl   0x0\(%eax,%eax,1\)
+
+0+b0 <nop4>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 40 00          	nopl   0x0\(%eax\)
+
+0+c0 <nop3>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 00             	nopl   \(%eax\)
+
+0+d0 <nop2>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 90                	xchg   %ax,%ax
+#pass
diff --git a/gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d b/gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d
new file mode 100644
index 0000000..99c3213
--- /dev/null
+++ b/gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d
@@ -0,0 +1,162 @@
+#as: -mtune=bdver2
+#source: nops-1.s
+#objdump: -drw
+#name: x86-64 -mtune=bdver2 nops 1
+
+.*: +file format .*
+
+
+Disassembly of section .text:
+
+0+ <nop15>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+[ 	]*[a-f0-9]+:	0f 1f 84 00 00 00 00 00 	nopl   0x0\(%rax,%rax,1\)
+
+0+10 <nop14>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+
+0+20 <nop13>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+
+0+30 <nop12>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+
+0+40 <nop11>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 44 00 00       	nopl   0x0\(%rax,%rax,1\)
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+
+0+50 <nop10>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 2e 0f 1f 84 00 00 00 00 00 	nopw   %cs:0x0\(%rax,%rax,1\)
+
+0+60 <nop9>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 84 00 00 00 00 00 	nopw   0x0\(%rax,%rax,1\)
+
+0+70 <nop8>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 84 00 00 00 00 00 	nopl   0x0\(%rax,%rax,1\)
+
+0+80 <nop7>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+
+0+90 <nop6>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+
+0+a0 <nop5>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 44 00 00       	nopl   0x0\(%rax,%rax,1\)
+
+0+b0 <nop4>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 40 00          	nopl   0x0\(%rax\)
+
+0+c0 <nop3>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 00             	nopl   \(%rax\)
+
+0+d0 <nop2>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 90                	xchg   %ax,%ax
+#pass
diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 04803db..06f9daf 100644
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,8 @@
+2011-05-04  Quentin Neill  <quentin.neill@amd.com>
+
+	* i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
+	* i386-init.h: Regenerated.
+
 2011-04-27  Nick Clifton  <nickc@redhat.com>
 
 	* po/da.po: Updated Danish translation.
diff --git a/opcodes/i386-gen.c b/opcodes/i386-gen.c
index d4adcf8..3d7de38 100644
--- a/opcodes/i386-gen.c
+++ b/opcodes/i386-gen.c
@@ -90,6 +90,8 @@ static initializer cpu_flag_init[] =
     "Cpu186|Cpu286|Cpu386|Cpu486|Cpu586|Cpu686|CpuSYSCALL|CpuRdtscp|Cpu387|Cpu687|CpuFISTTP|CpuNop|CpuMMX|Cpu3dnow|Cpu3dnowA|CpuSSE|CpuSSE2|CpuSSE3|CpuSSE4a|CpuABM|CpuLM" },
   { "CPU_BDVER1_FLAGS",
     "Cpu186|Cpu286|Cpu386|Cpu486|Cpu586|Cpu686|CpuSYSCALL|CpuRdtscp|Cpu387|Cpu687|CpuFISTTP|CpuNop|CpuMMX|CpuSSE|CpuSSE2|CpuSSE3|CpuSSE4a|CpuABM|CpuLM|CpuFMA4|CpuXOP|CpuLWP" },
+  { "CPU_BDVER2_FLAGS",
+    "Cpu186|Cpu286|Cpu386|Cpu486|Cpu586|Cpu686|CpuSYSCALL|CpuRdtscp|Cpu387|Cpu687|CpuFISTTP|CpuNop|CpuMMX|CpuSSE|CpuSSE2|CpuSSE3|CpuSSE4a|CpuABM|CpuLM|CpuFMA4|CpuXOP|CpuLWP|CpuBMI|CpuTBM" },
   { "CPU_8087_FLAGS",
     "Cpu8087" },
   { "CPU_287_FLAGS",
diff --git a/opcodes/i386-init.h b/opcodes/i386-init.h
index d8edd1a..6433596 100644
--- a/opcodes/i386-init.h
+++ b/opcodes/i386-init.h
@@ -139,6 +139,11 @@
       0, 0, 0, 0, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 0,  \
       0, 0, 0, 1, 0, 0, 0, 1, 0, 0, 0 } }
 
+#define CPU_BDVER2_FLAGS \
+  { { 1, 1, 1, 1, 1, 1, 0, 1, 1, 0, 0, 1, 1, 1, 1, 1, 1, 0, 0, 1,  \
+      0, 0, 0, 0, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1,  \
+      1, 0, 0, 1, 0, 0, 0, 1, 0, 0, 0 } }
+
 #define CPU_8087_FLAGS \
   { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  \
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  \
-- 
1.7.1


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

* Re: [PATCH] add bdver2 processor.
  2011-05-04 16:16 [PATCH] add bdver2 processor Quentin Neill
@ 2011-05-04 16:45 ` H.J. Lu
       [not found]   ` <BANLkTinukDoLD-yJ6SyfDbAjLO=NvHiN2w@mail.gmail.com>
  0 siblings, 1 reply; 12+ messages in thread
From: H.J. Lu @ 2011-05-04 16:45 UTC (permalink / raw)
  To: Quentin Neill; +Cc: binutils

On Wed, May 4, 2011 at 9:16 AM, Quentin Neill
<quentin.neill.gnu@gmail.com> wrote:
> This patch adds the bdver2 processor flag.  It passes make check on x86-64.
>
> Okay to commit?
> --
> Quentin
>
>
> gas/
>        * config/tc-i386.c (cpu_arch): Add bdver2.
>        (i386_align_code): Add PROCESSOR_BDVER2 cases.
>        * config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
>        * doc/c-i386.texi: Add bdver2.
>
> opcodes/
>        * i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
>        * i386-init.h: Regenerated.
>
> gas/testsuite/
>        * gas/i386/i386.exp: Add new bdver2 test cases.
>        * gas/i386/nops-1-bdver2.d: New.
>        * gas/i386/x86-64-nops-1-bdver2.d: New.
>

Do you nreally nned PROCESSOR_BDVER2? Can you use PROCESSOR_BDVER1? instead?

-- 
H.J.

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

* Re: [PATCH] add bdver2 processor.
       [not found]     ` <BANLkTinNnuhwjwp_0+FubD4=wzSixt6XvA@mail.gmail.com>
@ 2011-05-10 16:52       ` Quentin Neill
  2011-05-10 17:56         ` H.J. Lu
  0 siblings, 1 reply; 12+ messages in thread
From: Quentin Neill @ 2011-05-10 16:52 UTC (permalink / raw)
  To: H.J. Lu, binutils

On Fri, May 6, 2011 at 7:40 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Thu, May 5, 2011 at 9:09 PM, Quentin Neill
> <quentin.neill.gnu@gmail.com> wrote:
>> On Wed, May 4, 2011 at 11:45 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>> On Wed, May 4, 2011 at 9:16 AM, Quentin Neill
>>> <quentin.neill.gnu@gmail.com> wrote:
>>>> This patch adds the bdver2 processor flag.  It passes make check on x86-64.
>>>>
>>>> Okay to commit?
>>>> --
>>>> Quentin
>>>>
>>>>
>>>> gas/
>>>>        * config/tc-i386.c (cpu_arch): Add bdver2.
>>>>        (i386_align_code): Add PROCESSOR_BDVER2 cases.
>>>>        * config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
>>>>        * doc/c-i386.texi: Add bdver2.
>>>>
>>>> opcodes/
>>>>        * i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
>>>>        * i386-init.h: Regenerated.
>>>>
>>>> gas/testsuite/
>>>>        * gas/i386/i386.exp: Add new bdver2 test cases.
>>>>        * gas/i386/nops-1-bdver2.d: New.
>>>>        * gas/i386/x86-64-nops-1-bdver2.d: New.
>>>>
>>>
>>> Do you nreally nned PROCESSOR_BDVER2? Can you use PROCESSOR_BDVER1? instead?
>>>
>>> --
>>> H.J.
>>>
>>
>> BDVER2 adds BMI and TBM.
>
> That is controlled by CPU_BDVER2_FLAGS.
>

[ Sorry for the delay,  I was out for a couple of days.  Re-adding
binutils@sourceware.org (I meant to copy on my 2nd reply). ]

So without PROCESSOR_BDVER2, that means no -march=bdver2 flag support,
but what else?

What if -march/-mtune support is added to GCC for bdver2, how would it
convey -march=bdver2 to the assembler?
-- 
Quentin

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

* Re: [PATCH] add bdver2 processor.
  2011-05-10 16:52       ` Quentin Neill
@ 2011-05-10 17:56         ` H.J. Lu
  2011-05-10 18:28           ` Quentin Neill
  0 siblings, 1 reply; 12+ messages in thread
From: H.J. Lu @ 2011-05-10 17:56 UTC (permalink / raw)
  To: Quentin Neill; +Cc: binutils

On Tue, May 10, 2011 at 9:51 AM, Quentin Neill
<quentin.neill.gnu@gmail.com> wrote:
> On Fri, May 6, 2011 at 7:40 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>> On Thu, May 5, 2011 at 9:09 PM, Quentin Neill
>> <quentin.neill.gnu@gmail.com> wrote:
>>> On Wed, May 4, 2011 at 11:45 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>> On Wed, May 4, 2011 at 9:16 AM, Quentin Neill
>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>> This patch adds the bdver2 processor flag.  It passes make check on x86-64.
>>>>>
>>>>> Okay to commit?
>>>>> --
>>>>> Quentin
>>>>>
>>>>>
>>>>> gas/
>>>>>        * config/tc-i386.c (cpu_arch): Add bdver2.
>>>>>        (i386_align_code): Add PROCESSOR_BDVER2 cases.
>>>>>        * config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
>>>>>        * doc/c-i386.texi: Add bdver2.
>>>>>
>>>>> opcodes/
>>>>>        * i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
>>>>>        * i386-init.h: Regenerated.
>>>>>
>>>>> gas/testsuite/
>>>>>        * gas/i386/i386.exp: Add new bdver2 test cases.
>>>>>        * gas/i386/nops-1-bdver2.d: New.
>>>>>        * gas/i386/x86-64-nops-1-bdver2.d: New.
>>>>>
>>>>
>>>> Do you nreally nned PROCESSOR_BDVER2? Can you use PROCESSOR_BDVER1? instead?
>>>>
>>>> --
>>>> H.J.
>>>>
>>>
>>> BDVER2 adds BMI and TBM.
>>
>> That is controlled by CPU_BDVER2_FLAGS.
>>
>
> [ Sorry for the delay,  I was out for a couple of days.  Re-adding
> binutils@sourceware.org (I meant to copy on my 2nd reply). ]
>
> So without PROCESSOR_BDVER2, that means no -march=bdver2 flag support,
> but what else?
>

Can you replace PROCESSOR_BDVER2 with  PROCESSOR_BDVER1
to see if it makes a difference?

-- 
H.J.

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

* Re: [PATCH] add bdver2 processor.
  2011-05-10 17:56         ` H.J. Lu
@ 2011-05-10 18:28           ` Quentin Neill
  2011-05-10 19:11             ` H.J. Lu
  0 siblings, 1 reply; 12+ messages in thread
From: Quentin Neill @ 2011-05-10 18:28 UTC (permalink / raw)
  To: H.J. Lu, binutils

On Tue, May 10, 2011 at 12:55 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Tue, May 10, 2011 at 9:51 AM, Quentin Neill
> <quentin.neill.gnu@gmail.com> wrote:
>> On Fri, May 6, 2011 at 7:40 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>> On Thu, May 5, 2011 at 9:09 PM, Quentin Neill
>>> <quentin.neill.gnu@gmail.com> wrote:
>>>> On Wed, May 4, 2011 at 11:45 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>>> On Wed, May 4, 2011 at 9:16 AM, Quentin Neill
>>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>>> This patch adds the bdver2 processor flag.  It passes make check on x86-64.
>>>>>>
>>>>>> Okay to commit?
>>>>>> --
>>>>>> Quentin
>>>>>>
>>>>>>
>>>>>> gas/
>>>>>>        * config/tc-i386.c (cpu_arch): Add bdver2.
>>>>>>        (i386_align_code): Add PROCESSOR_BDVER2 cases.
>>>>>>        * config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
>>>>>>        * doc/c-i386.texi: Add bdver2.
>>>>>>
>>>>>> opcodes/
>>>>>>        * i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
>>>>>>        * i386-init.h: Regenerated.
>>>>>>
>>>>>> gas/testsuite/
>>>>>>        * gas/i386/i386.exp: Add new bdver2 test cases.
>>>>>>        * gas/i386/nops-1-bdver2.d: New.
>>>>>>        * gas/i386/x86-64-nops-1-bdver2.d: New.
>>>>>>
>>>>>
>>>>> Do you nreally nned PROCESSOR_BDVER2? Can you use PROCESSOR_BDVER1? instead?
>>>>>
>>>>> --
>>>>> H.J.
>>>>>
>>>>
>>>> BDVER2 adds BMI and TBM.
>>>
>>> That is controlled by CPU_BDVER2_FLAGS.
>>>
>>
>> [ Sorry for the delay,  I was out for a couple of days.  Re-adding
>> binutils@sourceware.org (I meant to copy on my 2nd reply). ]
>>
>> So without PROCESSOR_BDVER2, that means no -march=bdver2 flag support,
>> but what else?
>>
>
> Can you replace PROCESSOR_BDVER2 with  PROCESSOR_BDVER1
> to see if it makes a difference?
>
Now I understand, I was thinking you meant remove "bdver2" flag as well.

Should I rename the PROCESSOR_BDVER1 variable to perhaps PROCESSOR_BDVER?
-- 
Quentin

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

* Re: [PATCH] add bdver2 processor.
  2011-05-10 18:28           ` Quentin Neill
@ 2011-05-10 19:11             ` H.J. Lu
  2011-05-10 20:12               ` Quentin Neill
  2011-05-11  6:40               ` Jan Beulich
  0 siblings, 2 replies; 12+ messages in thread
From: H.J. Lu @ 2011-05-10 19:11 UTC (permalink / raw)
  To: Quentin Neill; +Cc: binutils

On Tue, May 10, 2011 at 11:28 AM, Quentin Neill
<quentin.neill.gnu@gmail.com> wrote:
> On Tue, May 10, 2011 at 12:55 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>> On Tue, May 10, 2011 at 9:51 AM, Quentin Neill
>> <quentin.neill.gnu@gmail.com> wrote:
>>> On Fri, May 6, 2011 at 7:40 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>> On Thu, May 5, 2011 at 9:09 PM, Quentin Neill
>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>> On Wed, May 4, 2011 at 11:45 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>>>> On Wed, May 4, 2011 at 9:16 AM, Quentin Neill
>>>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>>>> This patch adds the bdver2 processor flag.  It passes make check on x86-64.
>>>>>>>
>>>>>>> Okay to commit?
>>>>>>> --
>>>>>>> Quentin
>>>>>>>
>>>>>>>
>>>>>>> gas/
>>>>>>>        * config/tc-i386.c (cpu_arch): Add bdver2.
>>>>>>>        (i386_align_code): Add PROCESSOR_BDVER2 cases.
>>>>>>>        * config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
>>>>>>>        * doc/c-i386.texi: Add bdver2.
>>>>>>>
>>>>>>> opcodes/
>>>>>>>        * i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
>>>>>>>        * i386-init.h: Regenerated.
>>>>>>>
>>>>>>> gas/testsuite/
>>>>>>>        * gas/i386/i386.exp: Add new bdver2 test cases.
>>>>>>>        * gas/i386/nops-1-bdver2.d: New.
>>>>>>>        * gas/i386/x86-64-nops-1-bdver2.d: New.
>>>>>>>
>>>>>>
>>>>>> Do you nreally nned PROCESSOR_BDVER2? Can you use PROCESSOR_BDVER1? instead?
>>>>>>
>>>>>> --
>>>>>> H.J.
>>>>>>
>>>>>
>>>>> BDVER2 adds BMI and TBM.
>>>>
>>>> That is controlled by CPU_BDVER2_FLAGS.
>>>>
>>>
>>> [ Sorry for the delay,  I was out for a couple of days.  Re-adding
>>> binutils@sourceware.org (I meant to copy on my 2nd reply). ]
>>>
>>> So without PROCESSOR_BDVER2, that means no -march=bdver2 flag support,
>>> but what else?
>>>
>>
>> Can you replace PROCESSOR_BDVER2 with  PROCESSOR_BDVER1
>> to see if it makes a difference?
>>
> Now I understand, I was thinking you meant remove "bdver2" flag as well.
>
> Should I rename the PROCESSOR_BDVER1 variable to perhaps PROCESSOR_BDVER?

Sounds good to me.


-- 
H.J.

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

* Re: [PATCH] add bdver2 processor.
  2011-05-10 19:11             ` H.J. Lu
@ 2011-05-10 20:12               ` Quentin Neill
  2011-05-10 21:15                 ` H.J. Lu
  2011-05-11  6:40               ` Jan Beulich
  1 sibling, 1 reply; 12+ messages in thread
From: Quentin Neill @ 2011-05-10 20:12 UTC (permalink / raw)
  To: H.J. Lu; +Cc: binutils

[-- Attachment #1: Type: text/plain, Size: 2391 bytes --]

On Tue, May 10, 2011 at 2:10 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Tue, May 10, 2011 at 11:28 AM, Quentin Neill
> <quentin.neill.gnu@gmail.com> wrote:
>> On Tue, May 10, 2011 at 12:55 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>> On Tue, May 10, 2011 at 9:51 AM, Quentin Neill
>>> <quentin.neill.gnu@gmail.com> wrote:
>>>> On Fri, May 6, 2011 at 7:40 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>>> On Thu, May 5, 2011 at 9:09 PM, Quentin Neill
>>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>>> On Wed, May 4, 2011 at 11:45 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>>>>> On Wed, May 4, 2011 at 9:16 AM, Quentin Neill
>>>>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>>>>> This patch adds the bdver2 processor flag.  It passes make check on x86-64.
>>>>>>>>
>>>>>>>> Okay to commit?
>>>>>>>> --
>>>>>>>> Quentin
>>>>>>>>
>>>>>>>>
>>>>>>>> gas/
>>>>>>>>        * config/tc-i386.c (cpu_arch): Add bdver2.
>>>>>>>>        (i386_align_code): Add PROCESSOR_BDVER2 cases.
>>>>>>>>        * config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
>>>>>>>>        * doc/c-i386.texi: Add bdver2.
>>>>>>>>
>>>>>>>> opcodes/
>>>>>>>>        * i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
>>>>>>>>        * i386-init.h: Regenerated.
>>>>>>>>
>>>>>>>> gas/testsuite/
>>>>>>>>        * gas/i386/i386.exp: Add new bdver2 test cases.
>>>>>>>>        * gas/i386/nops-1-bdver2.d: New.
>>>>>>>>        * gas/i386/x86-64-nops-1-bdver2.d: New.
>>>>>>>>
>>>>>>>
>>>>>>> Do you nreally nned PROCESSOR_BDVER2? Can you use PROCESSOR_BDVER1? instead?
>>>>>>>
>>>>>>> --
>>>>>>> H.J.
>>>>>>>
>>>>>>
>>>>>> BDVER2 adds BMI and TBM.
>>>>>
>>>>> That is controlled by CPU_BDVER2_FLAGS.
>>>>>
>>>>
>>>> [ Sorry for the delay,  I was out for a couple of days.  Re-adding
>>>> binutils@sourceware.org (I meant to copy on my 2nd reply). ]
>>>>
>>>> So without PROCESSOR_BDVER2, that means no -march=bdver2 flag support,
>>>> but what else?
>>>>
>>>
>>> Can you replace PROCESSOR_BDVER2 with  PROCESSOR_BDVER1
>>> to see if it makes a difference?
>>>
>> Now I understand, I was thinking you meant remove "bdver2" flag as well.
>>
>> Should I rename the PROCESSOR_BDVER1 variable to perhaps PROCESSOR_BDVER?
>
> Sounds good to me.

Okay, the attached follows H.J.'s suggestions, and passes "make check"
in x86-64.

Okay to commit?
-- 
Quentin

[-- Attachment #2: 7053-2011_05_10-bdver2-quentin.neill_amd.com-patch.txt --]
[-- Type: text/plain, Size: 20781 bytes --]

From bc3df59306233070f6d3c2f1f52f514cdb32c014 Mon Sep 17 00:00:00 2001
From: Quentin Neill <quentin.neill@amd.com>
Date: Wed, 4 May 2011 09:39:03 -0500
Subject: [PATCH] 2011-05-10  Quentin Neill  <quentin.neill@amd.com>

gas/
	* config/tc-i386.c (cpu_arch): Add bdver2 and rename
	PROCESSOR_BDVER1 to PROCESSOR_BDVER.
	(i386_align_code): Rename PROCESSOR_BDVER1.
	(processor_type): Ditto.
	* doc/c-i386.texi: Add bdver2.

opcodes/
	* i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
	* i386-init.h: Regenerated.

gas/testsuite/
	* gas/i386/i386.exp: Add new bdver2 test cases.
	* gas/i386/nops-1-bdver2.d: New.
	* gas/i386/x86-64-nops-1-bdver2.d: New.
---
 gas/ChangeLog                                 |    8 ++
 gas/config/tc-i386.c                          |   10 +-
 gas/config/tc-i386.h                          |    2 +-
 gas/doc/c-i386.texi                           |    3 +-
 gas/testsuite/ChangeLog                       |    6 +
 gas/testsuite/gas/i386/i386.exp               |    2 +
 gas/testsuite/gas/i386/nops-1-bdver2.d        |  162 +++++++++++++++++++++++++
 gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d |  162 +++++++++++++++++++++++++
 opcodes/ChangeLog                             |    5 +
 opcodes/i386-gen.c                            |    2 +
 opcodes/i386-init.h                           |    5 +
 11 files changed, 361 insertions(+), 6 deletions(-)
 create mode 100644 gas/testsuite/gas/i386/nops-1-bdver2.d
 create mode 100644 gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d

diff --git a/gas/ChangeLog b/gas/ChangeLog
index e5f4ac5..56a0177 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,11 @@
+2011-05-10  Quentin Neill  <quentin.neill@amd.com>
+
+	* config/tc-i386.c (cpu_arch): Add bdver2 and rename
+	PROCESSOR_BDVER1 to PROCESSOR_BDVER.
+	(i386_align_code): Rename PROCESSOR_BDVER1.
+	(processor_type): Ditto.
+	* doc/c-i386.texi: Add bdver2.
+
 2011-05-10  Alan Modra  <amodra@gmail.com>
 
 	* dw2gencfi.c (tc_cfi_endproc): Avoid "set but not used" error.
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index 69ac150..d208a0d 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -645,8 +645,10 @@ static const arch_entry cpu_arch[] =
     CPU_K8_FLAGS, 0, 0 },
   { STRING_COMMA_LEN ("amdfam10"), PROCESSOR_AMDFAM10,
     CPU_AMDFAM10_FLAGS, 0, 0 },
-  { STRING_COMMA_LEN ("bdver1"), PROCESSOR_BDVER1,
+  { STRING_COMMA_LEN ("bdver1"), PROCESSOR_BDVER,
     CPU_BDVER1_FLAGS, 0, 0 },
+  { STRING_COMMA_LEN ("bdver2"), PROCESSOR_BDVER,
+    CPU_BDVER2_FLAGS, 0, 0 },
   { STRING_COMMA_LEN (".8087"), PROCESSOR_UNKNOWN,
     CPU_8087_FLAGS, 0, 0 },
   { STRING_COMMA_LEN (".287"), PROCESSOR_UNKNOWN,
@@ -1023,7 +1025,7 @@ i386_align_code (fragS *fragP, int count)
      PROCESSOR_CORE, PROCESSOR_CORE2, PROCESSOR_COREI7, and
      PROCESSOR_GENERIC64, alt_long_patt will be used.
      3. For PROCESSOR_ATHLON, PROCESSOR_K6, PROCESSOR_K8 and
-     PROCESSOR_AMDFAM10, and PROCESSOR_BDVER1, alt_short_patt
+     PROCESSOR_AMDFAM10, and PROCESSOR_BDVER, alt_short_patt
      will be used.
 
      When -mtune= isn't used, alt_long_patt will be used if
@@ -1076,7 +1078,7 @@ i386_align_code (fragS *fragP, int count)
 	    case PROCESSOR_ATHLON:
 	    case PROCESSOR_K8:
 	    case PROCESSOR_AMDFAM10:
-	    case PROCESSOR_BDVER1:
+	    case PROCESSOR_BDVER:
 	      patt = alt_short_patt;
 	      break;
 	    case PROCESSOR_I386:
@@ -1105,7 +1107,7 @@ i386_align_code (fragS *fragP, int count)
 	    case PROCESSOR_ATHLON:
 	    case PROCESSOR_K8:
 	    case PROCESSOR_AMDFAM10:
-	    case PROCESSOR_BDVER1:
+	    case PROCESSOR_BDVER:
 	    case PROCESSOR_GENERIC32:
 	      /* We use cpu_arch_isa_flags to check if we CAN optimize
 		 with nops.  */
diff --git a/gas/config/tc-i386.h b/gas/config/tc-i386.h
index bf227c6..bbe71e0 100644
--- a/gas/config/tc-i386.h
+++ b/gas/config/tc-i386.h
@@ -230,7 +230,7 @@ enum processor_type
   PROCESSOR_GENERIC32,
   PROCESSOR_GENERIC64,
   PROCESSOR_AMDFAM10,
-  PROCESSOR_BDVER1
+  PROCESSOR_BDVER
 };
 
 extern enum processor_type cpu_arch_tune;
diff --git a/gas/doc/c-i386.texi b/gas/doc/c-i386.texi
index 34a69b0..bedff79 100644
--- a/gas/doc/c-i386.texi
+++ b/gas/doc/c-i386.texi
@@ -118,6 +118,7 @@ processor names are recognized:
 @code{k8},
 @code{amdfam10},
 @code{bdver1},
+@code{bdver2},
 @code{generic32} and
 @code{generic64}.
 
@@ -1000,7 +1001,7 @@ supported on the CPU specified.  The choices for @var{cpu_type} are:
 @item @samp{prescott} @tab @samp{nocona} @tab @samp{core} @tab @samp{core2}
 @item @samp{corei7} @tab @samp{l1om}
 @item @samp{k6} @tab @samp{k6_2} @tab @samp{athlon} @tab @samp{k8}
-@item @samp{amdfam10} @tab @samp{bdver1}
+@item @samp{amdfam10} @tab @samp{bdver1} @tab @samp{bdver2}
 @item @samp{generic32} @tab @samp{generic64}
 @item @samp{.mmx} @tab @samp{.sse} @tab @samp{.sse2} @tab @samp{.sse3}
 @item @samp{.ssse3} @tab @samp{.sse4.1} @tab @samp{.sse4.2} @tab @samp{.sse4}
diff --git a/gas/testsuite/ChangeLog b/gas/testsuite/ChangeLog
index d5790a1..b1c7e61 100644
--- a/gas/testsuite/ChangeLog
+++ b/gas/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2010-05-10  Quentin Neill  <quentin.neill@amd.com>
+
+	* gas/i386/i386.exp: Add new bdver2 test cases.
+	* gas/i386/nops-1-bdver2.d: New.
+	* gas/i386/x86-64-nops-1-bdver2.d: New.
+
 2011-05-09  Paul Brook  <paul@codesourcery.com>
 
 	* gas/tic6x/unwind-1.d: New test.
diff --git a/gas/testsuite/gas/i386/i386.exp b/gas/testsuite/gas/i386/i386.exp
index 0eb151f..cc732f8 100644
--- a/gas/testsuite/gas/i386/i386.exp
+++ b/gas/testsuite/gas/i386/i386.exp
@@ -82,6 +82,7 @@ if [expr ([istarget "i*86-*-*"] ||  [istarget "x86_64-*-*"]) && [gas_32_check]]
     run_dump_test "nops-1-k8"
     run_dump_test "nops-1-core2"
     run_dump_test "nops-1-bdver1"
+    run_dump_test "nops-1-bdver2"
     run_dump_test "nops-2"
     run_dump_test "nops-2-i386"
     run_dump_test "nops-2-core2"
@@ -316,6 +317,7 @@ if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_64_check]] t
     run_dump_test "x86-64-nops-1-core2"
     run_dump_test "x86-64-nops-1-pentium"
     run_dump_test "x86-64-nops-1-bdver1"
+    run_dump_test "x86-64-nops-1-bdver2"
     run_dump_test "x86-64-nops-2"
     run_dump_test "x86-64-nops-3"
     run_dump_test "x86-64-nops-4"
diff --git a/gas/testsuite/gas/i386/nops-1-bdver2.d b/gas/testsuite/gas/i386/nops-1-bdver2.d
new file mode 100644
index 0000000..d33d6b7
--- /dev/null
+++ b/gas/testsuite/gas/i386/nops-1-bdver2.d
@@ -0,0 +1,162 @@
+#as: -mtune=bdver2
+#source: nops-1.s
+#objdump: -drw
+#name: i386 -mtune=bdver2 nops 1
+
+.*: +file format .*
+
+
+Disassembly of section .text:
+
+0+ <nop15>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+[ 	]*[a-f0-9]+:	0f 1f 84 00 00 00 00 00 	nopl   0x0\(%eax,%eax,1\)
+
+0+10 <nop14>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+
+0+20 <nop13>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+
+0+30 <nop12>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+
+0+40 <nop11>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 44 00 00       	nopl   0x0\(%eax,%eax,1\)
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+
+0+50 <nop10>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 2e 0f 1f 84 00 00 00 00 00 	nopw   %cs:0x0\(%eax,%eax,1\)
+
+0+60 <nop9>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 84 00 00 00 00 00 	nopw   0x0\(%eax,%eax,1\)
+
+0+70 <nop8>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 84 00 00 00 00 00 	nopl   0x0\(%eax,%eax,1\)
+
+0+80 <nop7>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%eax\)
+
+0+90 <nop6>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%eax,%eax,1\)
+
+0+a0 <nop5>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 44 00 00       	nopl   0x0\(%eax,%eax,1\)
+
+0+b0 <nop4>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 40 00          	nopl   0x0\(%eax\)
+
+0+c0 <nop3>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 00             	nopl   \(%eax\)
+
+0+d0 <nop2>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 90                	xchg   %ax,%ax
+#pass
diff --git a/gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d b/gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d
new file mode 100644
index 0000000..99c3213
--- /dev/null
+++ b/gas/testsuite/gas/i386/x86-64-nops-1-bdver2.d
@@ -0,0 +1,162 @@
+#as: -mtune=bdver2
+#source: nops-1.s
+#objdump: -drw
+#name: x86-64 -mtune=bdver2 nops 1
+
+.*: +file format .*
+
+
+Disassembly of section .text:
+
+0+ <nop15>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+[ 	]*[a-f0-9]+:	0f 1f 84 00 00 00 00 00 	nopl   0x0\(%rax,%rax,1\)
+
+0+10 <nop14>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+
+0+20 <nop13>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+
+0+30 <nop12>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+
+0+40 <nop11>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 44 00 00       	nopl   0x0\(%rax,%rax,1\)
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+
+0+50 <nop10>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 2e 0f 1f 84 00 00 00 00 00 	nopw   %cs:0x0\(%rax,%rax,1\)
+
+0+60 <nop9>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 84 00 00 00 00 00 	nopw   0x0\(%rax,%rax,1\)
+
+0+70 <nop8>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 84 00 00 00 00 00 	nopl   0x0\(%rax,%rax,1\)
+
+0+80 <nop7>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 80 00 00 00 00 	nopl   0x0\(%rax\)
+
+0+90 <nop6>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 0f 1f 44 00 00    	nopw   0x0\(%rax,%rax,1\)
+
+0+a0 <nop5>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 44 00 00       	nopl   0x0\(%rax,%rax,1\)
+
+0+b0 <nop4>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 40 00          	nopl   0x0\(%rax\)
+
+0+c0 <nop3>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	0f 1f 00             	nopl   \(%rax\)
+
+0+d0 <nop2>:
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	90                   	nop
+[ 	]*[a-f0-9]+:	66 90                	xchg   %ax,%ax
+#pass
diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 04803db..57a4f01 100644
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,8 @@
+2011-05-10  Quentin Neill  <quentin.neill@amd.com>
+
+	* i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
+	* i386-init.h: Regenerated.
+
 2011-04-27  Nick Clifton  <nickc@redhat.com>
 
 	* po/da.po: Updated Danish translation.
diff --git a/opcodes/i386-gen.c b/opcodes/i386-gen.c
index d4adcf8..3d7de38 100644
--- a/opcodes/i386-gen.c
+++ b/opcodes/i386-gen.c
@@ -90,6 +90,8 @@ static initializer cpu_flag_init[] =
     "Cpu186|Cpu286|Cpu386|Cpu486|Cpu586|Cpu686|CpuSYSCALL|CpuRdtscp|Cpu387|Cpu687|CpuFISTTP|CpuNop|CpuMMX|Cpu3dnow|Cpu3dnowA|CpuSSE|CpuSSE2|CpuSSE3|CpuSSE4a|CpuABM|CpuLM" },
   { "CPU_BDVER1_FLAGS",
     "Cpu186|Cpu286|Cpu386|Cpu486|Cpu586|Cpu686|CpuSYSCALL|CpuRdtscp|Cpu387|Cpu687|CpuFISTTP|CpuNop|CpuMMX|CpuSSE|CpuSSE2|CpuSSE3|CpuSSE4a|CpuABM|CpuLM|CpuFMA4|CpuXOP|CpuLWP" },
+  { "CPU_BDVER2_FLAGS",
+    "Cpu186|Cpu286|Cpu386|Cpu486|Cpu586|Cpu686|CpuSYSCALL|CpuRdtscp|Cpu387|Cpu687|CpuFISTTP|CpuNop|CpuMMX|CpuSSE|CpuSSE2|CpuSSE3|CpuSSE4a|CpuABM|CpuLM|CpuFMA4|CpuXOP|CpuLWP|CpuBMI|CpuTBM" },
   { "CPU_8087_FLAGS",
     "Cpu8087" },
   { "CPU_287_FLAGS",
diff --git a/opcodes/i386-init.h b/opcodes/i386-init.h
index d8edd1a..6433596 100644
--- a/opcodes/i386-init.h
+++ b/opcodes/i386-init.h
@@ -139,6 +139,11 @@
       0, 0, 0, 0, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 0,  \
       0, 0, 0, 1, 0, 0, 0, 1, 0, 0, 0 } }
 
+#define CPU_BDVER2_FLAGS \
+  { { 1, 1, 1, 1, 1, 1, 0, 1, 1, 0, 0, 1, 1, 1, 1, 1, 1, 0, 0, 1,  \
+      0, 0, 0, 0, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1,  \
+      1, 0, 0, 1, 0, 0, 0, 1, 0, 0, 0 } }
+
 #define CPU_8087_FLAGS \
   { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  \
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  \
-- 
1.7.1


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

* Re: [PATCH] add bdver2 processor.
  2011-05-10 20:12               ` Quentin Neill
@ 2011-05-10 21:15                 ` H.J. Lu
  0 siblings, 0 replies; 12+ messages in thread
From: H.J. Lu @ 2011-05-10 21:15 UTC (permalink / raw)
  To: Quentin Neill; +Cc: binutils

On Tue, May 10, 2011 at 1:11 PM, Quentin Neill
<quentin.neill.gnu@gmail.com> wrote:
> On Tue, May 10, 2011 at 2:10 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>> On Tue, May 10, 2011 at 11:28 AM, Quentin Neill
>> <quentin.neill.gnu@gmail.com> wrote:
>>> On Tue, May 10, 2011 at 12:55 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>> On Tue, May 10, 2011 at 9:51 AM, Quentin Neill
>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>> On Fri, May 6, 2011 at 7:40 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>>>> On Thu, May 5, 2011 at 9:09 PM, Quentin Neill
>>>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>>>> On Wed, May 4, 2011 at 11:45 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>>>>>>> On Wed, May 4, 2011 at 9:16 AM, Quentin Neill
>>>>>>>> <quentin.neill.gnu@gmail.com> wrote:
>>>>>>>>> This patch adds the bdver2 processor flag.  It passes make check on x86-64.
>>>>>>>>>
>>>>>>>>> Okay to commit?
>>>>>>>>> --
>>>>>>>>> Quentin
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> gas/
>>>>>>>>>        * config/tc-i386.c (cpu_arch): Add bdver2.
>>>>>>>>>        (i386_align_code): Add PROCESSOR_BDVER2 cases.
>>>>>>>>>        * config/tc-i386.h (processor_type): Add PROCESSOR_BDVER2.
>>>>>>>>>        * doc/c-i386.texi: Add bdver2.
>>>>>>>>>
>>>>>>>>> opcodes/
>>>>>>>>>        * i386-gen.c (cpu_flag_init): Add new CPU_BDVER2_FLAGS.
>>>>>>>>>        * i386-init.h: Regenerated.
>>>>>>>>>
>>>>>>>>> gas/testsuite/
>>>>>>>>>        * gas/i386/i386.exp: Add new bdver2 test cases.
>>>>>>>>>        * gas/i386/nops-1-bdver2.d: New.
>>>>>>>>>        * gas/i386/x86-64-nops-1-bdver2.d: New.
>>>>>>>>>
>>>>>>>>
>>>>>>>> Do you nreally nned PROCESSOR_BDVER2? Can you use PROCESSOR_BDVER1? instead?
>>>>>>>>
>>>>>>>> --
>>>>>>>> H.J.
>>>>>>>>
>>>>>>>
>>>>>>> BDVER2 adds BMI and TBM.
>>>>>>
>>>>>> That is controlled by CPU_BDVER2_FLAGS.
>>>>>>
>>>>>
>>>>> [ Sorry for the delay,  I was out for a couple of days.  Re-adding
>>>>> binutils@sourceware.org (I meant to copy on my 2nd reply). ]
>>>>>
>>>>> So without PROCESSOR_BDVER2, that means no -march=bdver2 flag support,
>>>>> but what else?
>>>>>
>>>>
>>>> Can you replace PROCESSOR_BDVER2 with  PROCESSOR_BDVER1
>>>> to see if it makes a difference?
>>>>
>>> Now I understand, I was thinking you meant remove "bdver2" flag as well.
>>>
>>> Should I rename the PROCESSOR_BDVER1 variable to perhaps PROCESSOR_BDVER?
>>
>> Sounds good to me.
>
> Okay, the attached follows H.J.'s suggestions, and passes "make check"
> in x86-64.
>
> Okay to commit?


OK.


-- 
H.J.

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

* Re: [PATCH] add bdver2 processor.
  2011-05-10 19:11             ` H.J. Lu
  2011-05-10 20:12               ` Quentin Neill
@ 2011-05-11  6:40               ` Jan Beulich
  2011-05-11 14:24                 ` Quentin Neill
  1 sibling, 1 reply; 12+ messages in thread
From: Jan Beulich @ 2011-05-11  6:40 UTC (permalink / raw)
  To: H.J. Lu, Quentin Neill; +Cc: binutils

>>> On 10.05.11 at 21:10, "H.J. Lu" <hjl.tools@gmail.com> wrote:
> On Tue, May 10, 2011 at 11:28 AM, Quentin Neill
>> Should I rename the PROCESSOR_BDVER1 variable to perhaps PROCESSOR_BDVER?
> 
> Sounds good to me.

And in a couple of years from now nobody will be able to tell what the
"VER" in the name stands for ...

Jan

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

* Re: [PATCH] add bdver2 processor.
  2011-05-11  6:40               ` Jan Beulich
@ 2011-05-11 14:24                 ` Quentin Neill
  2011-05-12  2:17                   ` Quentin Neill
  0 siblings, 1 reply; 12+ messages in thread
From: Quentin Neill @ 2011-05-11 14:24 UTC (permalink / raw)
  To: Jan Beulich; +Cc: H.J. Lu, binutils

On Wed, May 11, 2011 at 1:40 AM, Jan Beulich <JBeulich@novell.com> wrote:
>>>> On 10.05.11 at 21:10, "H.J. Lu" <hjl.tools@gmail.com> wrote:
>> On Tue, May 10, 2011 at 11:28 AM, Quentin Neill
>>> Should I rename the PROCESSOR_BDVER1 variable to perhaps PROCESSOR_BDVER?
>>
>> Sounds good to me.
>
> And in a couple of years from now nobody will be able to tell what the
> "VER" in the name stands for ...
>
> Jan

I thought of that last night when I got home.

I'll change it to PROCESSOR_BD with another patch.
-- 
Quentin

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

* Re: [PATCH] add bdver2 processor.
  2011-05-11 14:24                 ` Quentin Neill
@ 2011-05-12  2:17                   ` Quentin Neill
  2011-05-12  2:34                     ` H.J. Lu
  0 siblings, 1 reply; 12+ messages in thread
From: Quentin Neill @ 2011-05-12  2:17 UTC (permalink / raw)
  To: Jan Beulich; +Cc: H.J. Lu, binutils

[-- Attachment #1: Type: text/plain, Size: 899 bytes --]

On Wed, May 11, 2011 at 9:24 AM, Quentin Neill
<quentin.neill.gnu@gmail.com> wrote:
> On Wed, May 11, 2011 at 1:40 AM, Jan Beulich <JBeulich@novell.com> wrote:
>>>>> On 10.05.11 at 21:10, "H.J. Lu" <hjl.tools@gmail.com> wrote:
>>> On Tue, May 10, 2011 at 11:28 AM, Quentin Neill
>>>> Should I rename the PROCESSOR_BDVER1 variable to perhaps PROCESSOR_BDVER?
>>>
>>> Sounds good to me.
>>
>> And in a couple of years from now nobody will be able to tell what the
>> "VER" in the name stands for ...
>>
>> Jan
>
> I thought of that last night when I got home.
>
> I'll change it to PROCESSOR_BD with another patch.
> --
> Quentin

So attached is the tested PROCESSOR_BD variable rename patch, which
passes "make check" in x86_64.

Okay to commit?
-- 
Quentin

P.S. I noticed that on yesterday's cvs commit I neglected commit the
opcodes/ directory; ARG!   I went ahead and did that before this
patch.

[-- Attachment #2: 7055-2011_05_11-quentin_neill_amd_com-patch.txt --]
[-- Type: text/plain, Size: 2920 bytes --]

From 5cfb2a9d93899f7920b70306f5a1c12d543fcbc3 Mon Sep 17 00:00:00 2001
From: Quentin Neill <quentin.neill@amd.com>
Date: Wed, 11 May 2011 21:10:05 -0500
Subject: [PATCH] 2011-05-11  Quentin Neill  <quentin.neill@amd.com>

	* config/tc-i386.c (cpu_arch): Rename PROCESSOR_BDVER1 to PROCESSOR_BD.
	(i386_align_code): Ditto
---
 gas/ChangeLog        |    5 +++++
 gas/config/tc-i386.c |   10 +++++-----
 gas/config/tc-i386.h |    2 +-
 3 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 56a0177..db1aa3c 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2011-05-11  Quentin Neill  <quentin.neill@amd.com>
+
+	* config/tc-i386.c (cpu_arch): Rename PROCESSOR_BDVER1 to PROCESSOR_BD.
+	(i386_align_code): Ditto
+
 2011-05-10  Quentin Neill  <quentin.neill@amd.com>
 
 	* config/tc-i386.c (cpu_arch): Add bdver2 and rename
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index d208a0d..8bbf9db 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -645,9 +645,9 @@ static const arch_entry cpu_arch[] =
     CPU_K8_FLAGS, 0, 0 },
   { STRING_COMMA_LEN ("amdfam10"), PROCESSOR_AMDFAM10,
     CPU_AMDFAM10_FLAGS, 0, 0 },
-  { STRING_COMMA_LEN ("bdver1"), PROCESSOR_BDVER,
+  { STRING_COMMA_LEN ("bdver1"), PROCESSOR_BD,
     CPU_BDVER1_FLAGS, 0, 0 },
-  { STRING_COMMA_LEN ("bdver2"), PROCESSOR_BDVER,
+  { STRING_COMMA_LEN ("bdver2"), PROCESSOR_BD,
     CPU_BDVER2_FLAGS, 0, 0 },
   { STRING_COMMA_LEN (".8087"), PROCESSOR_UNKNOWN,
     CPU_8087_FLAGS, 0, 0 },
@@ -1025,7 +1025,7 @@ i386_align_code (fragS *fragP, int count)
      PROCESSOR_CORE, PROCESSOR_CORE2, PROCESSOR_COREI7, and
      PROCESSOR_GENERIC64, alt_long_patt will be used.
      3. For PROCESSOR_ATHLON, PROCESSOR_K6, PROCESSOR_K8 and
-     PROCESSOR_AMDFAM10, and PROCESSOR_BDVER, alt_short_patt
+     PROCESSOR_AMDFAM10, and PROCESSOR_BD, alt_short_patt
      will be used.
 
      When -mtune= isn't used, alt_long_patt will be used if
@@ -1078,7 +1078,7 @@ i386_align_code (fragS *fragP, int count)
 	    case PROCESSOR_ATHLON:
 	    case PROCESSOR_K8:
 	    case PROCESSOR_AMDFAM10:
-	    case PROCESSOR_BDVER:
+	    case PROCESSOR_BD:
 	      patt = alt_short_patt;
 	      break;
 	    case PROCESSOR_I386:
@@ -1107,7 +1107,7 @@ i386_align_code (fragS *fragP, int count)
 	    case PROCESSOR_ATHLON:
 	    case PROCESSOR_K8:
 	    case PROCESSOR_AMDFAM10:
-	    case PROCESSOR_BDVER:
+	    case PROCESSOR_BD:
 	    case PROCESSOR_GENERIC32:
 	      /* We use cpu_arch_isa_flags to check if we CAN optimize
 		 with nops.  */
diff --git a/gas/config/tc-i386.h b/gas/config/tc-i386.h
index bbe71e0..092e8c6 100644
--- a/gas/config/tc-i386.h
+++ b/gas/config/tc-i386.h
@@ -230,7 +230,7 @@ enum processor_type
   PROCESSOR_GENERIC32,
   PROCESSOR_GENERIC64,
   PROCESSOR_AMDFAM10,
-  PROCESSOR_BDVER
+  PROCESSOR_BD
 };
 
 extern enum processor_type cpu_arch_tune;
-- 
1.7.1


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

* Re: [PATCH] add bdver2 processor.
  2011-05-12  2:17                   ` Quentin Neill
@ 2011-05-12  2:34                     ` H.J. Lu
  0 siblings, 0 replies; 12+ messages in thread
From: H.J. Lu @ 2011-05-12  2:34 UTC (permalink / raw)
  To: Quentin Neill; +Cc: Jan Beulich, binutils

On Wed, May 11, 2011 at 7:16 PM, Quentin Neill
<quentin.neill.gnu@gmail.com> wrote:
> On Wed, May 11, 2011 at 9:24 AM, Quentin Neill
> <quentin.neill.gnu@gmail.com> wrote:
>> On Wed, May 11, 2011 at 1:40 AM, Jan Beulich <JBeulich@novell.com> wrote:
>>>>>> On 10.05.11 at 21:10, "H.J. Lu" <hjl.tools@gmail.com> wrote:
>>>> On Tue, May 10, 2011 at 11:28 AM, Quentin Neill
>>>>> Should I rename the PROCESSOR_BDVER1 variable to perhaps PROCESSOR_BDVER?
>>>>
>>>> Sounds good to me.
>>>
>>> And in a couple of years from now nobody will be able to tell what the
>>> "VER" in the name stands for ...
>>>
>>> Jan
>>
>> I thought of that last night when I got home.
>>
>> I'll change it to PROCESSOR_BD with another patch.
>> --
>> Quentin
>
> So attached is the tested PROCESSOR_BD variable rename patch, which
> passes "make check" in x86_64.
>
> Okay to commit?

OK.


-- 
H.J.

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

end of thread, other threads:[~2011-05-12  2:34 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-04 16:16 [PATCH] add bdver2 processor Quentin Neill
2011-05-04 16:45 ` H.J. Lu
     [not found]   ` <BANLkTinukDoLD-yJ6SyfDbAjLO=NvHiN2w@mail.gmail.com>
     [not found]     ` <BANLkTinNnuhwjwp_0+FubD4=wzSixt6XvA@mail.gmail.com>
2011-05-10 16:52       ` Quentin Neill
2011-05-10 17:56         ` H.J. Lu
2011-05-10 18:28           ` Quentin Neill
2011-05-10 19:11             ` H.J. Lu
2011-05-10 20:12               ` Quentin Neill
2011-05-10 21:15                 ` H.J. Lu
2011-05-11  6:40               ` Jan Beulich
2011-05-11 14:24                 ` Quentin Neill
2011-05-12  2:17                   ` Quentin Neill
2011-05-12  2:34                     ` H.J. Lu

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).