https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #14 from Jakub Jelinek ---
One thing against adding these builtins (especially when we have a type-generic
builtin which can handle all types, not just some of them) is that clang has
added them completely unorthogonally.
GCC has
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
Xi Ruoyao changed:
What|Removed |Added
CC||xry111 at gcc dot gnu.org
--- Comment #13
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #12 from Andrew Pinski ---
(In reply to Sam James from comment #11)
> (In reply to Andrew Pinski from comment #7)
> > Sounds like that HPC QCD app is broken. You should report it to the
> > developers. Also since __builtin_ctzg is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #11 from Sam James ---
(In reply to Andrew Pinski from comment #7)
> Sounds like that HPC QCD app is broken. You should report it to the
> developers. Also since __builtin_ctzg is added, adding __builtin_ctzs really
> does not make
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #10 from Andrew Pinski ---
(In reply to gnzlbg from comment #8)
> Oh! Sorry! I see this was resolved as WONTFIX.
>
> Does that mean that GCC, in contrast with the LLVM community - which is
> always super helpful and friendly when
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #9 from gnzlbg ---
> > Look at that part where __builtin_ctzs is mentioned there.
> It just says it is valid inside a constexpr and nothing else about the
> builtin.
Yes, seems like a bug / pr should be opened in the llvm project
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #8 from gnzlbg ---
Oh! Sorry! I see this was resolved as WONTFIX.
Does that mean that GCC, in contrast with the LLVM community - which is always
super helpful and friendly when it comes to trying to enable their toolchains
to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #7 from Andrew Pinski ---
(In reply to gnzlbg from comment #6)
> Thanks for all the quick feedback!
>
> > Also clang does not even document __builtin_ctzs anywhere ...
>
> This builtin is documented in the list of clang bultins.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #6 from gnzlbg ---
Thanks for all the quick feedback!
> Also clang does not even document __builtin_ctzs anywhere ...
This builtin is documented in the list of clang bultins. You can find it by
using CTRL+F for the builtin, in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #4 from Andrew Pinski ---
So just for future reference __builtin_c[lt]zs is for short.
But with GCC 14 (trunk) added __builtin_ct[lt]g, there is no need to add
__builtin_c[lt]zs to GCC.
You could just do:
#if
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #3 from Andrew Pinski ---
Also clang does not even document __builtin_ctzs anywhere ...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
Andrew Pinski changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
--- Comment #1 from Andrew Pinski ---
ok, and?
14 matches
Mail list logo