https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #20 from Segher Boessenkool ---
There were various issues with the builtins. We could have made them work
for just (say) power8 and up, in time for GCC 8, but then for a long time no
one would be able to use __builtin_fcti[wd] on oth
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #19 from Jakub Jelinek ---
The main issue of the builtins that have been reverted was that it was too tied
to a particular HW instruction, if the hw instruction doesn't exist, don't
support it (and this wasn't working properly, it was
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #18 from Steven Munroe ---
(In reply to jos...@codesourcery.com from comment #17)
> And, when long is 64-bit, there is no corresponding standard function to
> round to 32-bit integer with "invalid" raised for out-of-range results -
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #17 from joseph at codesourcery dot com ---
And, when long is 64-bit, there is no corresponding standard function to
round to 32-bit integer with "invalid" raised for out-of-range results -
but there is (undocumented) __builtin_irin
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #16 from joseph at codesourcery dot com ---
These instructions use the current rounding mode, not round-to-zero.
That is, they correspond to the lrint / llrint functions, given
-fno-math-errno.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #15 from Jakub Jelinek ---
Why aren't just C cast from float/double to int/long enough?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #14 from Bill Schmidt ---
The functions are broken, and a release is imminent. This will be revisited in
the next release, and backports considered at least for Advance Toolchain.
Time and resources are finite...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #13 from Steven Munroe ---
WTF which part of requirement did you not understand.
You you should implement the direct moves (to GPRs) to complete the
__builtin_fctid and __builtin_fctiw implementation.
But to just remove them is mis
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
Carl Love changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #11 from Steven Munroe ---
The requirement was to reduce the use of (in-line) assembler in libraries. Asm
is error prone in the light of 32/64-bit ABI difference and the compiler
(usual) generates the correct code for the target.
Flo
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #10 from Carl Love ---
These builtins were per a request from Steve Monroe. Not sure why he wanted
them or if he actually ever used them.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #9 from Carl Love ---
This test case is in the list in PR 84422 . Working on a patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
Segher Boessenkool changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassig
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #7 from Segher Boessenkool ---
We actually do *not* need TARGET_FPRND; fctiw and fctid are ISA 1.xx insns.
Before power8 we do not allow putting SImode in FPRs.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #6 from Jakub Jelinek ---
Seems any -O* level with -mcpu=power6 or -mcpu=power7 actually.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #5 from Jakub Jelinek ---
The ICE is with -m64 -mcpu=power6 -O0.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
Jakub Jelinek changed:
What|Removed |Added
CC||carll at gcc dot gnu.org
--- Comment #4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
--- Comment #3 from Segher Boessenkool ---
No, many other builtins are also problematic (there are a few more PRs open).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
Richard Biener changed:
What|Removed |Added
Version|unknown |8.0
Target Milestone|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
Segher Boessenkool changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever confirmed|0
22 matches
Mail list logo