[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 Jonathan Wakely changed: What|Removed |Added Target Milestone|13.0|12.3 --- Comment #18 from Jonathan Wa

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #17 from CVS Commits --- The releases/gcc-12 branch has been updated by Jonathan Wakely : https://gcc.gnu.org/g:8caf5805ad76125b84430b8653003f4776489d46 commit r12-9462-g8caf5805ad76125b84430b8653003f4776489d46 Author: Jonathan Wak

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #16 from Jonathan Wakely --- Unfortunately this change causes a regression for libs that were statically linked to libstdc++.a before the PR 91057 fix. Any object which has the buggy std::locale::id::_M_id() code linked into it can g

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 Jonathan Wakely changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #14 from Jonathan Wakely --- Should be fixed now, I think

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #13 from CVS Commits --- The master branch has been updated by Jonathan Wakely : https://gcc.gnu.org/g:a7f51059fb009dcd7d491d6b2164bce75dbd9975 commit r13-3917-ga7f51059fb009dcd7d491d6b2164bce75dbd9975 Author: Jonathan Wakely Date

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 Jonathan Wakely changed: What|Removed |Added CC|jwakely.gcc at gmail dot com | --- Comment #12 from Jonathan

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #11 from Jonathan Wakely --- As I said in Bug 107632 comment 2: I'm kinda tempted to just disable the new optimization on these targets, the handling of compat facets for different float ABIs is impossible to get right.

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 seurer at gcc dot gnu.org changed: What|Removed |Added CC||seurer at gcc dot gnu.org --

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #9 from Jonathan Wakely --- This causes an ABI regression on powerpc64le: FAIL: libstdc++-abi/abi_check

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #8 from CVS Commits --- The master branch has been updated by Jonathan Wakely : https://gcc.gnu.org/g:b3ac43a3c05744d62a963d656bed782fc867ad79 commit r13-3888-gb3ac43a3c05744d62a963d656bed782fc867ad79 Author: Jonathan Wakely Date:

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 Jonathan Wakely changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |redi at gcc dot gnu.org

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 Jonathan Wakely changed: What|Removed |Added Target Milestone|--- |13.0

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #7 from Jonathan Wakely --- (In reply to Dmitry Prokoptsev from comment #6) > That would also work, I suppose (it even outperforms my original approach by > a tiny bit -- 33 ns for v2 vs 36 for my original implementation). > > There

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #6 from Dmitry Prokoptsev --- That would also work, I suppose (it even outperforms my original approach by a tiny bit -- 33 ns for v2 vs 36 for my original implementation). There are a few build errors in the alternative implementat

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #5 from Dmitry Prokoptsev --- Created attachment 52029 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52029&action=edit Build fix for alternative implementation v2.

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #4 from Jonathan Wakely --- (In reply to Jonathan Wakely from comment #3) > Created attachment 52025 [details] > Alternative implementation v2 > > The diagnostic regression is easy to fix with a static assertion before > calling __t

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 Jonathan Wakely changed: What|Removed |Added Attachment #52024|0 |1 is obsolete|

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 --- Comment #2 from Jonathan Wakely --- Created attachment 52024 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52024&action=edit Alternative implementation This seems like a much simpler approach. This causes 22_locale/ctype/is/string/8

[Bug libstdc++/103755] {has,use}_facet() and iostream constructor performance

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103755 Jonathan Wakely changed: What|Removed |Added Ever confirmed|0 |1 Severity|normal