diff options
| author | Alan Modra <amodra@gmail.com> | 2023-02-16 20:47:44 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2023-02-17 08:29:00 +1030 |
| commit | 4d826e132ccc38c3081209bf3a7a107f8b23ffd4 (patch) | |
| tree | 0e435bc2d145167b1347d781fa7cbb583094cc4b /gas | |
| parent | 12d67b37cde0d6f8492bc2052b3fe8da4691f9d0 (diff) | |
| download | gdb-4d826e132ccc38c3081209bf3a7a107f8b23ffd4.zip gdb-4d826e132ccc38c3081209bf3a7a107f8b23ffd4.tar.gz gdb-4d826e132ccc38c3081209bf3a7a107f8b23ffd4.tar.bz2 | |
PR30046, power cmpi leads to unknown architecture
PowerPC ELF always uses bfd_arch_powerpc, so we shouldn't allow the
gas -mpwr, -mpwr2 or -mpwrx options to choose bfd_arch_rs6000.
Given the possible values of ppc_cpu, I think the as_fatal at the end
of ppc_arch will never be reached, so it can be deleted and the code
simplified a little.
PR 30046
* config/tc-ppc.c (ppc_arch): Return bfd_arch_powerpc for ELF.
Delete dead code.
Diffstat (limited to 'gas')
| -rw-r--r-- | gas/config/tc-ppc.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/gas/config/tc-ppc.c b/gas/config/tc-ppc.c index 9450fa7..a3af5f5 100644 --- a/gas/config/tc-ppc.c +++ b/gas/config/tc-ppc.c @@ -1496,9 +1496,11 @@ ppc_set_cpu (void) enum bfd_architecture ppc_arch (void) { - const char *default_cpu = TARGET_CPU; ppc_set_cpu (); +#ifdef OBJ_ELF + return bfd_arch_powerpc; +#else if ((ppc_cpu & PPC_OPCODE_PPC) != 0) return bfd_arch_powerpc; if ((ppc_cpu & PPC_OPCODE_VLE) != 0) @@ -1507,14 +1509,12 @@ ppc_arch (void) return bfd_arch_rs6000; if ((ppc_cpu & (PPC_OPCODE_COMMON | PPC_OPCODE_ANY)) != 0) { - if (strcmp (default_cpu, "rs6000") == 0) - return bfd_arch_rs6000; - else if (startswith (default_cpu, "powerpc")) + const char *default_cpu = TARGET_CPU; + if (startswith (default_cpu, "powerpc")) return bfd_arch_powerpc; } - - as_fatal (_("neither Power nor PowerPC opcodes were selected.")); - return bfd_arch_unknown; + return bfd_arch_rs6000; +#endif } unsigned long |