[PATCH] D77924: ld128 demangle: allow space for 'L' suffix.

Elliott Hughes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 09:49:40 PDT 2020


enh added a comment.

In D77924#2228309 <https://reviews.llvm.org/D77924#2228309>, @RKSimon wrote:

> @enh This is causing a test failure on the windows/aarch64 cross compile bot, please can you take a look:
>
> http://lab.llvm.org:8011/builders/llvm-clang-win-x-aarch64/builds/2267
> http://lab.llvm.org:8011/builders/llvm-clang-win-x-aarch64/builds/2267/steps/test-libc%2B%2Babi/logs/FAIL%3A%20libc%2B%2Babi%3A%3A%20test_demangle.pass.cpp

is it just me or is the stdout logging from just before that assertion missing?

t.tmp.exe: C:/buildbot/as-builder-2/clang-win-x-aarch64/llvm-project/libcxxabi/test/test_demangle.pass.cpp:30001: void testFPLiterals(): Assertion `false' failed.

any idea how i can see the actual failure details?

          if (std::find(e_beg, e_end, demang) == e_end)
          {
              std::cout << fpCase->mangled << " -> " << fpCase->expecting[0] << '\
  n';
              std::cout << "Got instead: " << demang << '\n';
              assert(false);
              continue;
          }


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77924/new/

https://reviews.llvm.org/D77924



More information about the llvm-commits mailing list