Hello Joe McDonnell, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20721
to look at the new patch set (#2).
Change subject: IMPALA-12563: Fix UBSAN on ARM
......................................................................
IMPALA-12563: Fix UBSAN on ARM
Links gcc after all other libraries when building with UBSAN. On ARM,
several symbols are included that aren't present in libclang_rt (enabled
by -rtlib=compiler-rt for UBSAN builds) or in libgcc_s.so (needed with
the alternate rtlib); linking libgcc.a after all other libraries ensures
the symbols are present. There may be other repercussions, so this is
only done for UBSAN builds.
Skips FE tests with UBSAN on ARM due to increased use of thread-local
storage on ARM that exceeds some implementation-defined limit.
Setting '-XX:ThreadStackSize=16m' didn't help.
Change-Id: I799bedd1cc73c852b0edb928dc71166e534918ba
---
M CMakeLists.txt
M bin/run-all-tests.sh
M cmake_modules/clang_toolchain.cmake
3 files changed, 23 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/21/20721/2
--
To view, visit http://gerrit.cloudera.org:8080/20721
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I799bedd1cc73c852b0edb928dc71166e534918ba
Gerrit-Change-Number: 20721
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>