[Bug c++/98327] C++ Module ICE on Linux

2021-01-05 Thread nathan at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98327

Nathan Sidwell  changed:

   What|Removed |Added

   Keywords|ice-on-invalid-code |ice-on-valid-code
 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from Nathan Sidwell  ---
Diagnostic adjusted
 6ffaffd5d10 2021-01-05 | [c++]: Improve module-decl diagnostics [PR 98327]

[Bug c++/98327] C++ Module ICE on Linux

2021-01-05 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98327

--- Comment #5 from Martin Liška  ---
> I suspect Martin failed to copy the first line of the testcase 'module;' 
> That's important -- it's not an English introduction of a piece of code :)
> 

Yep, I failed to do that :)

Anyway, now I can bisect that it was fixed with r11-6309-g8be20b9e0d6d53d8.

[Bug c++/98327] C++ Module ICE on Linux

2021-01-04 Thread nathan at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98327

--- Comment #4 from Nathan Sidwell  ---
I cannot reproduce the reported crash -- like I said, I think this is fixed
(thanks for the backtrace).

I suspect Martin failed to copy the first line of the testcase 'module;' 
That's important -- it's not an English introduction of a piece of code :)

The error message could be improved to say where these decls are permitted,
rather than 'computer says no'.

[Bug c++/98327] C++ Module ICE on Linux

2021-01-04 Thread nathan at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98327

Nathan Sidwell  changed:

   What|Removed |Added

 Status|WAITING |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |nathan at gcc dot 
gnu.org

--- Comment #3 from Nathan Sidwell  ---
three things.
a) I think this is now fixed

b) it's not invalid code

c) Martin, I think you're falling foul of some of the requirements of where you
place code that is in the GMF (the bit between 'module;' and '[export] module
foo;'  But I think the diagnostic you're getting is not helping there.

[Bug c++/98327] C++ Module ICE on Linux

2020-12-17 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98327

Martin Liška  changed:

   What|Removed |Added

   Last reconfirmed||2020-12-17
 CC||marxin at gcc dot gnu.org
 Status|UNCONFIRMED |WAITING
 Ever confirmed|0   |1

--- Comment #2 from Martin Liška  ---
Can't reproduce with the current master:

$ g++ pr98327.cc -c -fmodules-ts -std=c++20
pr98327.cc:3:8: error: module-declaration not permitted here
3 | export module hello;
  |^~
pr98327.cc:4:1: error: ‘export’ may only occur after a module interface
declaration
4 | export inline void greeter (std::string_view name) noexcept
  | ^~

[Bug c++/98327] C++ Module ICE on Linux

2020-12-16 Thread euloanty at live dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98327

fdlbxtqi  changed:

   What|Removed |Added

 CC||nathan at gcc dot gnu.org

--- Comment #1 from fdlbxtqi  ---
module;
#include 
#include 
export module hello;
export inline void greeter (std::string_view name) noexcept
{
  std::cout << "Hello " << name << "!\n";
}