This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: [PATCH] Pretty printer test fixes and improvements
- From: Christophe Lyon <christophe dot lyon at linaro dot org>
- To: Jonathan Wakely <jwakely at redhat dot com>
- Cc: libstdc++ at gcc dot gnu dot org, gcc Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 9 Jan 2019 11:09:17 +0100
- Subject: Re: [PATCH] Pretty printer test fixes and improvements
- References: <20190108231529.GA18761@redhat.com>
On Wed, 9 Jan 2019 at 00:16, Jonathan Wakely <jwakely@redhat.com> wrote:
>
> Test that StdUniquePtrPrinter correctly prints std::unique_ptr objects
> using the old layout, prior to the PR libstdc++/77990 changes.
>
> The printer test for a valueless std::variant started to fail because
> the PR libstdc++/87431 fix meant it no longer became valueless. Change
> the test to use a type that is not trivially copyable, so that the
> exception causes it to become valueless.
>
> * testsuite/libstdc++-prettyprinters/compat.cc: Test printer support
> for old std::unique_ptr layout.
> * testsuite/libstdc++-prettyprinters/cxx17.cc: Fix std::variant test
> to become valueless. Add filesystem::path tests.
>
> Tested x86_64-linux, committed to trunk.
>
Hi Jonathan,
After this patch, I've noticed a new failure on aarch64:
FAIL: libstdc++-prettyprinters/cxx17.cc print p2
The log contains:
PASS: libstdc++-prettyprinters/cxx17.cc print p1
$27 = filesystem::path "/dir/." = {[root-directory] = "/", [1] =
"dir", [2] = "."}
got: $27 = filesystem::path "/dir/." = {[root-directory] = "/", [1] =
"dir", [2] = "."}
FAIL: libstdc++-prettyprinters/cxx17.cc print p2
Note that a number of cxx17.cc tests already failed on aarch64, and still fail:
FAIL: libstdc++-prettyprinters/cxx17.cc print p
FAIL: libstdc++-prettyprinters/cxx17.cc print wp
FAIL: libstdc++-prettyprinters/cxx17.cc print q
FAIL: libstdc++-prettyprinters/cxx17.cc print wq
I've also noticed that the same test names appear twice, I think this confuses
test_summary etc...
Christophe