[PATCH] D67847: [Support] make report_fatal_error `abort` instead of `exit`

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 29 11:49:37 PST 2020


rnk added a comment.

If we can reland this patch, it should fix this new clang bug:
https://bugs.llvm.org/show_bug.cgi?id=44705

This was the list of failures you noted on the commit:

In rG647c3f4e47de8a850ffcaa897db68702d8d2459a#885042 <https://reviews.llvm.org/rG647c3f4e47de8a850ffcaa897db68702d8d2459a#885042>, @ychen wrote:

> This is going deeper...
>
> - Host compiler crash (looks like ppc64be-specific) http://lab.llvm.org:8011/builders/clang-ppc64be-linux-multistage/builds/23016 http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/43016 http://lab.llvm.org:8011/builders/clang-ppc64be-linux-lnt/builds/34286
> - clang/test/CXX/temp/temp.arg/temp.arg.template/p3-2a.cpp becomes pass/fail randomly
> - test timeout http://lab.llvm.org:8011/builders/openmp-gcc-x86_64-linux-debian/builds/2170/steps/test-openmp/logs/stdio
> - ThinLTO test failures on Redhat (passes on my ubuntu 18.04) http://lab.llvm.org:8011/builders/clang-ppc64le-rhel/builds/592
>
>   Will investigate.


The ppc64be stuff seems like only someone with hardware will be able to debug it.
For `clang/test/CXX/temp/temp.arg/temp.arg.template/p3-2a.cpp`, you are saying you can reproduce that locally?
The timeout seems like it could be flakiness or something unrelated.
The thinlto failures... I'm not sure either, but again it's PPC64, so it could be flakiness.

It seems like it would be best to alert the owners of those bots and reland and get new results, assuming things pass on less exotic bot configurations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67847





More information about the cfe-commits mailing list