[Bug libstdc++/81706] std::sin vectorization bug

2017-12-15 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 Jakub Jelinek changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug libstdc++/81706] std::sin vectorization bug

2017-10-25 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #9 from Jakub Jelinek --- Author: jakub Date: Wed Oct 25 08:05:58 2017 New Revision: 254069 URL: https://gcc.gnu.org/viewcvs?rev=254069&root=gcc&view=rev Log: PR libstdc++/81706 * attribs.c (attribute_value_equal): Us

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #8 from Jonathan Wakely --- (In reply to Richard Biener from comment #1) > The C library sin is vectorized by being appropriately annotated with OpenMP > SIMD attributes. This isn't the case for the libstdc++ variant which ends > up

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #7 from rguenther at suse dot de --- On Fri, 4 Aug 2017, jakub at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 > > --- Comment #6 from Jakub Jelinek --- > (In reply to rguent...@suse.de from comment #

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #6 from Jakub Jelinek --- (In reply to rguent...@suse.de from comment #5) > True, but it removes the ability to avoid the vectorized variant with > using the __builtin_ variant ;) Do we ever want to generate less efficient code when

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #5 from rguenther at suse dot de --- On Fri, 4 Aug 2017, jakub at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 > > --- Comment #4 from Jakub Jelinek --- > The C/C++ FE change would be something like:

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #4 from Jakub Jelinek --- The C/C++ FE change would be something like: --- gcc/tree.c.jj 2017-07-29 09:48:40.0 +0200 +++ gcc/tree.c 2017-08-04 12:06:35.636072718 +0200 @@ -5022,8 +5022,8 @@ attribute_value_equal (const_

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #2 from Jakub Jelinek --- Created attachment 41923 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41923&action=edit gcc8-pr81706.patch The reason for why cmath does this is that C++ wants to have sin overloads. And the problem

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #2 from Jakub Jelinek --- Created attachment 41923 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41923&action=edit gcc8-pr81706.patch The reason for why cmath does this is that C++ wants to have sin overloads. And the problem

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 Richard Biener changed: What|Removed |Added Keywords||missed-optimization Status|