Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-10-19 Thread Andrew Stubbs
On 18/10/11 15:42, Andrew Stubbs wrote: Which still jumps to not_found without closing f. Hmmm, I know I fixed that, I know I did! But I appear to have lost the change somewhere when I updated my checkout? I'll fix it now. Fixed and committed as attached. Apologies for the cock-up. :(

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-10-18 Thread Andrew Stubbs
On 17/10/11 14:24, Richard Earnshaw wrote: There's a presumption in host_detect_local_cpu() that CPU implementer will appear before CPU part in the output of /proc/cpuinfo. That's probably a pretty safe assumption (and it appears that it will handle that case relatively safely -- ie not

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-10-18 Thread Richard Earnshaw
On 18/10/11 15:23, Andrew Stubbs wrote: + /* Detect arch/cpu. */ + if (strncmp (buf, CPU part, sizeof (CPU part) - 1) == 0) + { + int i; + + if (cpu_table == NULL) + goto not_found; + Which still jumps to not_found without closing f. R.

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-10-18 Thread Andrew Stubbs
On 18/10/11 15:34, Richard Earnshaw wrote: On 18/10/11 15:23, Andrew Stubbs wrote: + /* Detect arch/cpu. */ + if (strncmp (buf, CPU part, sizeof (CPU part) - 1) == 0) + { + int i; + + if (cpu_table == NULL) + goto not_found; + Which still jumps to

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-10-17 Thread Richard Earnshaw
On 20/09/11 11:51, Andrew Stubbs wrote: On 09/09/11 12:55, Richard Earnshaw wrote: The part number field is meaningless outside of the context of a a specific vendor -- only taken as a pair can they refer to a specific part. So why is the vendor field hard-coded rather than factored into the

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-10-13 Thread Andrew Stubbs
Ping. On 20/09/11 11:51, Andrew Stubbs wrote: On 09/09/11 12:55, Richard Earnshaw wrote: The part number field is meaningless outside of the context of a a specific vendor -- only taken as a pair can they refer to a specific part. So why is the vendor field hard-coded rather than factored into

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-09-20 Thread Andrew Stubbs
On 09/09/11 12:55, Richard Earnshaw wrote: The part number field is meaningless outside of the context of a a specific vendor -- only taken as a pair can they refer to a specific part. So why is the vendor field hard-coded rather than factored into the table of parts. Maybe it would be better

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-09-09 Thread Richard Earnshaw
On 06/09/11 14:35, Andrew Stubbs wrote: This update adds many more magic numbers for various ARM CPUs, and also ensures that the implementer is ARM (as opposed to Marvell, etc.). The list is far from comprehensive, but it should cover many (but by no means all) of the cores in current use

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-09-06 Thread Andrew Stubbs
This update adds many more magic numbers for various ARM CPUs, and also ensures that the implementer is ARM (as opposed to Marvell, etc.). The list is far from comprehensive, but it should cover many (but by no means all) of the cores in current use and it would not be hard to add support for

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-08-30 Thread Stubbs, Andrew
On 29/08/11 04:29, Michael Hope wrote: On Sat, Aug 27, 2011 at 3:19 AM, Andrew Stubbsa...@codesourcery.com wrote: Hi all, This patch adds support for -mcpu=native, -mtune=native, and -march=native for ARM Linux hosts. So far, it only recognises Cortex-A8 and Cortex-A9, so I really need to

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-08-30 Thread Michael Hope
On Tue, Aug 30, 2011 at 10:47 PM, Stubbs, Andrew andrew_stu...@mentor.com wrote: On 29/08/11 04:29, Michael Hope wrote: On Sat, Aug 27, 2011 at 3:19 AM, Andrew Stubbsa...@codesourcery.com  wrote: Hi all, This patch adds support for -mcpu=native, -mtune=native, and -march=native for ARM Linux

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-08-28 Thread Michael Hope
On Sat, Aug 27, 2011 at 3:19 AM, Andrew Stubbs a...@codesourcery.com wrote: Hi all, This patch adds support for -mcpu=native, -mtune=native, and -march=native for ARM Linux hosts. So far, it only recognises Cortex-A8 and Cortex-A9, so I really need to find out what the magic part numbers

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-08-27 Thread Andrew Stubbs
On 26/08/11 17:16, Joseph S. Myers wrote: arm-tables.opt is a generated file. You need to modify the source files and regenerate it, not modify the generated file. Fixed; the native option value is now defined in arm.opt. Thanks for spotting this. OK? Andrew 2011-08-27 Andrew Stubbs

Re: [PATCH][ARM] -m{cpu,tune,arch}=native

2011-08-26 Thread Joseph S. Myers
On Fri, 26 Aug 2011, Andrew Stubbs wrote: Hi all, This patch adds support for -mcpu=native, -mtune=native, and -march=native for ARM Linux hosts. So far, it only recognises Cortex-A8 and Cortex-A9, so I really need to find out what the magic part numbers are for other cpus before this