c++: Don't synthesize sfk_comparison method multiple times [PR94907]
authorJakub Jelinek <jakub@redhat.com>
Wed, 6 May 2020 21:36:31 +0000 (23:36 +0200)
committerJakub Jelinek <jakub@redhat.com>
Wed, 6 May 2020 21:36:31 +0000 (23:36 +0200)
commit25ee2155ead87a5ea1c152a29341ee1e3275d706
tree13883e8b157af22e2ba0e308c9793a746035e25b
parent72a54e5e8157508c87594d1a6b53de90b76524ce
c++: Don't synthesize sfk_comparison method multiple times [PR94907]

On the following testcase we ICE, because synthesize_method is called twice
on the same sfk_comparison method fndecl, the first time it works fine
because start_preparsed_function in that case sets both
current_function_decl and cfun, but second time it is called it only sets
the former and keeps cfun NULL, so we ICE when trying to store
current_function_returns_value.
I think it is just wrong to call synthesize_method multiple times, and most
synthesize_method callers avoid that by not calling it if DECL_INITIAL is
already set, so this patch does that too.

2020-05-06  Jakub Jelinek  <jakub@redhat.com>

PR c++/94907
* method.c (defaulted_late_check): Don't call synthesize_method
on constexpr sfk_comparison if it has been called on it already.

* g++.dg/cpp2a/spaceship-synth8.C: New test.
gcc/cp/ChangeLog
gcc/cp/method.c
gcc/testsuite/ChangeLog
gcc/testsuite/g++.dg/cpp2a/spaceship-synth8.C [new file with mode: 0644]