On 7/10/20 12:33 PM, Ian Lance Taylor wrote:
As discussed in PR 96143, the C++ frontend and the demangler disagree
as to whether a lambda is a substitution candidate. According to the
ABI, the C++ frontend is behaving correctly. This patch changes the
demangler accordingly.
This caused the demangling of several existing test cases to change.
These test cases are long and complicated, and I frankly did not try
to understand whether the change is correct or not. I believe that
the change is correct, and at least these symbols continue to
demangle.
Did a full bootstrap and testsuite run on x86_64-pc-linux-gnu.
OK for mainline?
OK, thanks.
Ian
2020-07-10 Ian Lance Taylor <i...@golang.org>
PR demangler/96143
* cp-demangle.c (d_lambda): Don't add substitution candidate.
* testsuite/demangle-expected: Update a few existing test cases
accordingly, and add a new test case.
