* [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
[parent not found: <BANLkTinukDoLD-yJ6SyfDbAjLO=NvHiN2w@mail.gmail.com>]
[parent not found: <BANLkTinNnuhwjwp_0+FubD4=wzSixt6XvA@mail.gmail.com>]
* 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).