[PATCH] D86621: [clang][Sparc] Default to -mcpu=v9 for SparcV8 on Solaris
Rainer Orth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 27 02:17:12 PDT 2020
ro added a comment.
While doing 2-stage builds, the 2 `XFAIL`s due to Bug 42493 were required. However, I've since also done a 1-stage build with gcc 10, which shows
Unexpectedly Passed Tests (2):
UBSan-Standalone-sparc :: TestCases/Float/cast-overflow.cpp
UBSan-Standalone-sparc :: TestCases/Misc/log-path_test.cpp
which is no wonder since the underlying bug only exists in `clang` `long double` handling.
I'm a bit uncertain how to deal with this: since the Solaris/sparcv9 buildbot currently does 1-stage builds with gcc, the resulting `XPASS`es would turn the bot red. However, currently there's no clang-vs.-gcc feature in `lit` to distinguish the two builds, AFAICT. `libcxx` has something like this, but that might be overkill to extract.
Perhaps just remove the `XFAIL`s for now?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86621/new/
https://reviews.llvm.org/D86621
More information about the cfe-commits
mailing list