https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #17 from Segher Boessenkool ---
(In reply to Franz Sirl from comment #15)
> > "7400" and "403" are not supported target attribute values, fwiw (says the
> > manual).
>
> Hmm, I don't understand what you mean.
I mean that I cannot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #16 from Franz Sirl ---
Created attachment 51199
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51199=edit
Patch version with minimum changes against GCC10
This is the minimum version of the patch, it fixes this PR but still
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #15 from Franz Sirl ---
(In reply to Segher Boessenkool from comment #14)
> (In reply to Franz Sirl from comment #12)
> > The emitted .machine is easy to fix, what's not so easy to fix is the
> > intention behind Segher's change
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #14 from Segher Boessenkool ---
(In reply to Franz Sirl from comment #12)
> The emitted .machine is easy to fix, what's not so easy to fix is the
> intention behind Segher's change that caused the wrong .machine.
>
> Consider this
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
Segher Boessenkool changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |segher at gcc dot
gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #12 from Franz Sirl ---
The emitted .machine is easy to fix, what's not so easy to fix is the intention
behind Segher's change that caused the wrong .machine.
Consider this source compiled with -mcpu=7400:
void ppcaltivecfunc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #11 from Alan Modra ---
Preserving certain -m gas options goes back to this patch:
https://sourceware.org/pipermail/binutils/2008-January/054935.html
Given the number of ppc micros around with differing functional units, it is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #10 from Segher Boessenkool ---
(In reply to Franz Sirl from comment #9)
> (In reply to Segher Boessenkool from comment #8)
> > I don't think it is a good idea to add workaround upon workaround to avoid
> > some of the not-so-useful
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #9 from Franz Sirl ---
(In reply to Segher Boessenkool from comment #8)
> I don't think it is a good idea to add workaround upon workaround to avoid
> some of the not-so-useful behaviours of -many. Instead, we should just
> not use
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
Segher Boessenkool changed:
What|Removed |Added
CC||amodra at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
Franz Sirl changed:
What|Removed |Added
Attachment #51164|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #6 from Segher Boessenkool ---
(In reply to Franz Sirl from comment #4)
> How about something along this patch? It's not fully done (no good idea
> about SPEC stuff like "mcpu=7400: -mppc %{!mvsx:%{!maltivec:-maltivec}};"
> yet),
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #5 from Segher Boessenkool ---
(In reply to Franz Sirl from comment #3)
> (In reply to Segher Boessenkool from comment #1)
> > The -many is problematic, that is the whole point of this. As in this
> > example: on different
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #4 from Franz Sirl ---
Created attachment 51164
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51164=edit
Half-baken trial patch
How about something along this patch? It's not fully done (no good idea about
SPEC stuff like
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #3 from Franz Sirl ---
(In reply to Segher Boessenkool from comment #1)
> The -many is problematic, that is the whole point of this. As in this
> example: on different subtargets there are different machine code
> translations for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
--- Comment #2 from Segher Boessenkool ---
A good solution would add all those historical targets to
rs6000_machine_from_flags.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101393
Segher Boessenkool changed:
What|Removed |Added
CC||segher at gcc dot gnu.org
---
17 matches
Mail list logo
