* [Bug libstdc++/102499] Noexcept not matching fgor std::filesystem::path. Compilation fails for clang
2021-09-27 15:58 [Bug libstdc++/102499] New: Noexcept not matching fgor std::filesystem::path. Compilation fails for clang unlvsur at live dot com
@ 2021-09-27 16:03 ` unlvsur at live dot com
2021-09-27 21:08 ` [Bug libstdc++/102499] Noexcept not matching for " redi at gcc dot gnu.org
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: unlvsur at live dot com @ 2021-09-27 16:03 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102499
--- Comment #1 from cqwrteur <unlvsur at live dot com> ---
noexcept is missing for begin() end(). That leads to problem that breaks clang.
I manually modify the header file and it works.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug libstdc++/102499] Noexcept not matching for std::filesystem::path. Compilation fails for clang
2021-09-27 15:58 [Bug libstdc++/102499] New: Noexcept not matching fgor std::filesystem::path. Compilation fails for clang unlvsur at live dot com
2021-09-27 16:03 ` [Bug libstdc++/102499] " unlvsur at live dot com
@ 2021-09-27 21:08 ` redi at gcc dot gnu.org
2021-09-27 21:09 ` [Bug libstdc++/102499] [12 Regression] " redi at gcc dot gnu.org
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2021-09-27 21:08 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102499
Jonathan Wakely <redi at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|unassigned at gcc dot gnu.org |redi at gcc dot gnu.org
Status|UNCONFIRMED |ASSIGNED
Ever confirmed|0 |1
Last reconfirmed| |2021-09-27
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug libstdc++/102499] [12 Regression] Noexcept not matching for std::filesystem::path. Compilation fails for clang
2021-09-27 15:58 [Bug libstdc++/102499] New: Noexcept not matching fgor std::filesystem::path. Compilation fails for clang unlvsur at live dot com
2021-09-27 16:03 ` [Bug libstdc++/102499] " unlvsur at live dot com
2021-09-27 21:08 ` [Bug libstdc++/102499] Noexcept not matching for " redi at gcc dot gnu.org
@ 2021-09-27 21:09 ` redi at gcc dot gnu.org
2021-09-27 23:24 ` unlvsur at live dot com
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2021-09-27 21:09 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102499
Jonathan Wakely <redi at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Known to work| |11.2.1
Target Milestone|--- |12.0
Known to fail| |12.0
Summary|Noexcept not matching for |[12 Regression] Noexcept
|std::filesystem::path. |not matching for
|Compilation fails for clang |std::filesystem::path.
| |Compilation fails for clang
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug libstdc++/102499] [12 Regression] Noexcept not matching for std::filesystem::path. Compilation fails for clang
2021-09-27 15:58 [Bug libstdc++/102499] New: Noexcept not matching fgor std::filesystem::path. Compilation fails for clang unlvsur at live dot com
` (2 preceding siblings ...)
2021-09-27 21:09 ` [Bug libstdc++/102499] [12 Regression] " redi at gcc dot gnu.org
@ 2021-09-27 23:24 ` unlvsur at live dot com
2021-09-27 23:59 ` redi at gcc dot gnu.org
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: unlvsur at live dot com @ 2021-09-27 23:24 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102499
--- Comment #2 from cqwrteur <unlvsur at live dot com> ---
(In reply to cqwrteur from comment #1)
> noexcept is missing for begin() end(). That leads to problem that breaks
> clang.
>
> I manually modify the header file and it works.
https://github.com/gcc-mirror/gcc/commit/42eff613d0c10f88dc7a44b14981876401a09981#diff-8cb9eb19d74e89b3acffd1a56c8bf7ca102f342b3b18134bba7729e343bdbce9
It looks like this cleanup patch has some issues. Probably need a new test file
for clang.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug libstdc++/102499] [12 Regression] Noexcept not matching for std::filesystem::path. Compilation fails for clang
2021-09-27 15:58 [Bug libstdc++/102499] New: Noexcept not matching fgor std::filesystem::path. Compilation fails for clang unlvsur at live dot com
` (3 preceding siblings ...)
2021-09-27 23:24 ` unlvsur at live dot com
@ 2021-09-27 23:59 ` redi at gcc dot gnu.org
2021-09-28 0:07 ` unlvsur at live dot com
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2021-09-27 23:59 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102499
--- Comment #3 from Jonathan Wakely <redi at gcc dot gnu.org> ---
There are no tests using clang.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug libstdc++/102499] [12 Regression] Noexcept not matching for std::filesystem::path. Compilation fails for clang
2021-09-27 15:58 [Bug libstdc++/102499] New: Noexcept not matching fgor std::filesystem::path. Compilation fails for clang unlvsur at live dot com
` (4 preceding siblings ...)
2021-09-27 23:59 ` redi at gcc dot gnu.org
@ 2021-09-28 0:07 ` unlvsur at live dot com
2021-09-28 17:08 ` cvs-commit at gcc dot gnu.org
2021-09-28 17:10 ` redi at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: unlvsur at live dot com @ 2021-09-28 0:07 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102499
--- Comment #4 from cqwrteur <unlvsur at live dot com> ---
(In reply to Jonathan Wakely from comment #3)
> There are no tests using clang.
okay. i will report if I still see any issues with clang after the patch.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug libstdc++/102499] [12 Regression] Noexcept not matching for std::filesystem::path. Compilation fails for clang
2021-09-27 15:58 [Bug libstdc++/102499] New: Noexcept not matching fgor std::filesystem::path. Compilation fails for clang unlvsur at live dot com
` (5 preceding siblings ...)
2021-09-28 0:07 ` unlvsur at live dot com
@ 2021-09-28 17:08 ` cvs-commit at gcc dot gnu.org
2021-09-28 17:10 ` redi at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2021-09-28 17:08 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102499
--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jonathan Wakely <redi@gcc.gnu.org>:
https://gcc.gnu.org/g:f2b7f56a15d9cbbd2f0db22e0e39c4dd161bab69
commit r12-3930-gf2b7f56a15d9cbbd2f0db22e0e39c4dd161bab69
Author: Jonathan Wakely <jwakely@redhat.com>
Date: Mon Sep 27 22:07:12 2021 +0100
libstdc++: Fix mismatched noexcept-specifiers in filesystem::path
[PR102499]
Signed-off-by: Jonathan Wakely <jwakely@redhat.com>
libstdc++-v3/ChangeLog:
PR libstdc++/102499
* include/bits/fs_path.h (path::begin, path::end): Add noexcept
to declarations, to match definitions.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug libstdc++/102499] [12 Regression] Noexcept not matching for std::filesystem::path. Compilation fails for clang
2021-09-27 15:58 [Bug libstdc++/102499] New: Noexcept not matching fgor std::filesystem::path. Compilation fails for clang unlvsur at live dot com
` (6 preceding siblings ...)
2021-09-28 17:08 ` cvs-commit at gcc dot gnu.org
@ 2021-09-28 17:10 ` redi at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2021-09-28 17:10 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102499
Jonathan Wakely <redi at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|ASSIGNED |RESOLVED
--- Comment #6 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Fixed now, thanks for the report.
I do have local tests that I run with Clang, but they're not automated and not
part of the upstream testsuite, so I didn't notice this bug. My local clang
tests do pass again now.
^ permalink raw reply [flat|nested] 9+ messages in thread