[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[bergner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

Peter Bergner  changed:

   What|Removed |Added

 Status|RESOLVED|CLOSED

--- Comment #9 from Peter Bergner  ---
Fixed in trunk and the FSF 7 release branch.  Closing.

[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[bergner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

--- Comment #8 from Peter Bergner  ---
Author: bergner
Date: Mon Feb  5 19:17:37 2018
New Revision: 257392

URL: https://gcc.gnu.org/viewcvs?rev=257392=gcc=rev
Log:
Back port from mainline
2018-02-01  Peter Bergner  

PR target/56010
PR target/83743
* config/rs6000/driver-rs6000.c: #include "diagnostic.h".
#include "opts.h".
(rs6000_supported_cpu_names): New static variable.
(linux_cpu_translation_table): Likewise.
(elf_platform) : Define new static variable and use it.
Translate kernel AT_PLATFORM name to canonical name if needed.
Error if platform name is unknown.

Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/config/rs6000/driver-rs6000.c

[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[bergner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

Peter Bergner  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
URL|https://gcc.gnu.org/ml/gcc- |https://gcc.gnu.org/ml/gcc-
   |patches/2018-01/msg02059.ht |patches/2018-01/msg02469.ht
   |ml  |ml
 Resolution|--- |FIXED

--- Comment #7 from Peter Bergner  ---
Fixed on trunk.  Will wait a few days and then will back port it to FSF 7.

[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[bergner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

--- Comment #6 from Peter Bergner  ---
Author: bergner
Date: Thu Feb  1 18:26:51 2018
New Revision: 257305

URL: https://gcc.gnu.org/viewcvs?rev=257305=gcc=rev
Log:
PR target/56010
PR target/83743
* config/rs6000/driver-rs6000.c: #include "diagnostic.h".
#include "opts.h".
(rs6000_supported_cpu_names): New static variable.
(linux_cpu_translation_table): Likewise.
(elf_platform) : Define new static variable and use it.
Translate kernel AT_PLATFORM name to canonical name if needed.
Error if platform name is unknown.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/rs6000/driver-rs6000.c

[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[bergner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

Peter Bergner  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
URL||https://gcc.gnu.org/ml/gcc-
   ||patches/2018-01/msg02059.ht
   ||ml
   Last reconfirmed||2018-01-24
   Assignee|unassigned at gcc dot gnu.org  |bergner at gcc dot 
gnu.org
 Ever confirmed|0   |1

--- Comment #5 from Peter Bergner  ---
Patch submitted to fix this and PR56010.

[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[segher at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

--- Comment #4 from Segher Boessenkool  ---
Which is perfectly fine, except the error message is a bit confusing if you
don't know what's going on.

[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[bergner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

--- Comment #3 from Peter Bergner  ---
(In reply to Segher Boessenkool from comment #2)
> The driver asks the kernel, and the kernel knows.  It's just a string.

Correct, the kernel passes a string to the driver and the driver blindly uses
it as its -mcpu= option, without first determining whether it recognizes (ie,
supports it) or not.

[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[segher at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

Segher Boessenkool  changed:

   What|Removed |Added

 CC||segher at gcc dot gnu.org

--- Comment #2 from Segher Boessenkool  ---
The driver asks the kernel, and the kernel knows.  It's just a string.

[Bug target/83743] -mcpu=native causes gcc to exit in error if cpu is not recognized

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83743

--- Comment #1 from Richard Biener  ---
How does the "old" driver know power9?  I think mixing driver and compiler
versions isn't supported.