thurston added a comment.
This patch might have broke the buildbots, starting with when it was first
built in https://lab.llvm.org/buildbot/#/builders/85/builds/18390
/b/sanitizer-x86_64-linux-bootstrap-ubsan/build/libcxx_build_ubsan/include/c++/v1/stdlib.h:114:25:
runtime error: negation of -9223372036854775808 cannot be represented in type
'long'; cast to an unsigned type to negate this value to itself
#0 0x5586b6dfb9aa in abs
/b/sanitizer-x86_64-linux-bootstrap-ubsan/build/libcxx_build_ubsan/include/c++/v1/stdlib.h:114:10
#1 0x5586b6dfb9aa in (anonymous
namespace)::GNUELFDumper<llvm::object::ELFType<(llvm::support::endianness)1,
true>>::printRelRelaReloc((anonymous
namespace)::Relocation<llvm::object::ELFType<(llvm::support::endianness)1,
true>> const&, (anonymous
namespace)::RelSymbol<llvm::object::ELFType<(llvm::support::endianness)1,
true>> const&)
/b/sanitizer-x86_64-linux-bootstrap-ubsan/build/llvm-project/llvm/tools/llvm-readobj/ELFDumper.cpp:3778:21
(which is good insofar as it finds a real bug, but nonetheless is turning the
buildbots red)
In the latest buildbots, the error persists (along with an unrelated failure):
https://lab.llvm.org/buildbot/#/builders/85/builds/18413
The compiler code is a tad too nuanced for me to fix with confidence (e.g., my
first instinct would be to widen the type to int128 but I don't know if that
will break compatibility).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156821/new/
https://reviews.llvm.org/D156821
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
