[Bug c++/103118] [modules] ICE tree check in get_merge_kind at cp/module.cc

[johelegp at gmail dot com via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103118

Johel Ernesto Guerrero Peña  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #5 from Johel Ernesto Guerrero Peña  ---
Fixed somewhere along the way.

[Bug c++/103118] [modules] ICE tree check in get_merge_kind at cp/module.cc

[johelegp at gmail dot com via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103118

--- Comment #4 from Johel Ernesto Guerrero Peña  ---
With the setup in Comment 3, now I can also include `` in the GMF of a
module. I don't think this worked last week. Though all I'm doing in the module
is specializing a my-library type trait on some chrono date types.

[Bug c++/103118] [modules] ICE tree check in get_merge_kind at cp/module.cc

[johelegp at gmail dot com via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103118

--- Comment #3 from Johel Ernesto Guerrero Peña  ---
What works is including them in a non-module unit translation unit.

```C++
#if defined(__GNUC__) and not defined(__clang__)
#  include  // Workaround GCC bug 103118.
#  include// Workaround GCC bug 100583.
#endif
import waarudo.geometries;
import waarudo.numbers;
import waarudo.quantities;
#include 
```

[Bug c++/103118] [modules] ICE tree check in get_merge_kind at cp/module.cc

[johelegp at gmail dot com via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103118

--- Comment #2 from Johel Ernesto Guerrero Peña  ---
Or not. I forgot to uncomment uses of the chrono library.

[Bug c++/103118] [modules] ICE tree check in get_merge_kind at cp/module.cc

[johelegp at gmail dot com via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103118

--- Comment #1 from Johel Ernesto Guerrero Peña  ---
To work around this
- Do not include ``.
- Explicitly import a module that exports its definitions when you need to use
the chrono library.

I use the headers above to compile a header unit, which I export from a module
to work around the lack of a standard modules. Apparently, `` is
indirectly included, as using the chrono library works.