[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread swanson at ukfsn dot org
--- Additional Comments From swanson at ukfsn dot org 2008-10-12 10:20 --- Created an attachment (id=2997) --> (http://sourceware.org/bugzilla/attachment.cgi?id=2997&action=view) binutils-cvs-fix2-i386-nop.patch Add bitfield checking back to patch for .arch settings. First point, do y

[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread swanson at ukfsn dot org
--- Additional Comments From swanson at ukfsn dot org 2008-10-12 09:37 --- But currently the only arch flags checked in i386_align_code was cpui686 which is where we should disable use of NOPL for mentioned reasons. -- http://sourceware.org/bugzilla/show_bug.cgi?id=6957 --- You

[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread hjl dot tools at gmail dot com
--- Additional Comments From hjl dot tools at gmail dot com 2008-10-12 09:52 --- By default, assembler assumes CPU takes all instructions supported by assembler. You can use -march=XXX to limit instructions to only those supported by XXX. -- http://sourceware.org/bugzilla/show_bug.cg

[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread hjl dot tools at gmail dot com
--- Additional Comments From hjl dot tools at gmail dot com 2008-10-12 09:49 --- (In reply to comment #3) > But currently the only arch flags checked in i386_align_code was cpui686 which > is where we should disable use of NOPL for mentioned reasons. If you want to run the binary on CPU

[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread hjl dot tools at gmail dot com
--- Additional Comments From hjl dot tools at gmail dot com 2008-10-12 12:44 --- (In reply to comment #6) > Created an attachment (id=2997) --> (http://sourceware.org/bugzilla/attachment.cgi?id=2997&action=view) > binutils-cvs-fix2-i386-nop.patch > > Add bitfield checking back to patch

[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread hjl dot tools at gmail dot com
--- Additional Comments From hjl dot tools at gmail dot com 2008-10-12 12:45 --- (In reply to comment #6) > Created an attachment (id=2997) --> (http://sourceware.org/bugzilla/attachment.cgi?id=2997&action=view) > binutils-cvs-fix2-i386-nop.patch > > Add bitfield checking back to patch

[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread swanson at ukfsn dot org
--- Additional Comments From swanson at ukfsn dot org 2008-10-12 13:25 --- Sorry but how can PROCESSOR_UNKNOWN mean it can run any instructions? It's unknown! Therefore we must use lowest common denominator to expect it to work on all CPUs. Tune means scheduling. Arch means instructions

[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread hjl dot tools at gmail dot com
--- Additional Comments From hjl dot tools at gmail dot com 2008-10-12 21:30 --- (In reply to comment #9) > Sorry but how can PROCESSOR_UNKNOWN mean it can run any instructions? It's > unknown! Therefore we must use lowest common denominator to expect it to work > on > all CPUs. If we

[Bug gas/6957] i386 NOPs must be derived from march not mtune

2008-10-12 Thread hjl dot tools at gmail dot com
--- Additional Comments From hjl dot tools at gmail dot com 2008-10-12 21:35 --- This testcase works for me: [EMAIL PROTECTED] tmp]$ cat nops.s .text nop: movsbl %al,%esi .p2align 4 [EMAIL PROTECTED] tmp]$ as --32 -o nops.o nops.s -march=i486 -mtune=i686 [EMAIL P