[PATCH] D38277: [compiler-rt][CMake] Fix configuration on PowerPC with sanitizers

2017-09-29 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added inline comments. Comment at: compiler-rt/trunk/cmake/base-config-ix.cmake:154 + cmake_push_check_state() + string(REPLACE "-nodefaultlibs" "" CMAKE_REQUIRED_FLAGS ${OLD_CMAKE_REQUIRED_FLAGS}) TEST_BIG_ENDIAN(HOST_IS_BIG_ENDIAN) al

[PATCH] D38277: [compiler-rt][CMake] Fix configuration on PowerPC with sanitizers

2017-09-29 Thread Aleksey Shlyapnikov via Phabricator via cfe-commits
alekseyshl added inline comments. Comment at: compiler-rt/trunk/cmake/base-config-ix.cmake:154 + cmake_push_check_state() + string(REPLACE "-nodefaultlibs" "" CMAKE_REQUIRED_FLAGS ${OLD_CMAKE_REQUIRED_FLAGS}) TEST_BIG_ENDIAN(HOST_IS_BIG_ENDIAN)

[PATCH] D38277: [compiler-rt][CMake] Fix configuration on PowerPC with sanitizers

2017-09-29 Thread Aleksey Shlyapnikov via Phabricator via cfe-commits
alekseyshl added a comment. http://lab.llvm.org:8011/builders/sanitizer-ppc64be-linux/builds/4004 is broken by this change Repository: rL LLVM https://reviews.llvm.org/D38277 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

[PATCH] D38277: [compiler-rt][CMake] Fix configuration on PowerPC with sanitizers

2017-09-29 Thread Jonas Hahnfeld via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314512: [CMake] Fix configuration on PowerPC with sanitizers (authored by Hahnfeld). Changed prior to commit: https://reviews.llvm.org/D38277?vs=116977&id=117132#toc Repository: rL LLVM https://revi

[PATCH] D38277: [compiler-rt][CMake] Fix configuration on PowerPC with sanitizers

2017-09-28 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld updated this revision to Diff 116977. Hahnfeld retitled this revision from "[compiler-rt}[CMake] Fix configuration on PowerPC with sanitizers" to "[compiler-rt][CMake] Fix configuration on PowerPC with sanitizers". Hahnfeld added a subscriber: gtbercea. https://reviews.llvm.org/D38277

[PATCH] D38277: [compiler-rt}[CMake] Fix configuration on PowerPC with sanitizers

2017-09-28 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld marked 2 inline comments as done. Hahnfeld added a comment. The error with `-DLLVM_USE_SANITIZER=Address` is -- Check if the system is big endian -- Searching 16 bit integer -- Looking for stddef.h -- Looking for stddef.h - not found -- Check size of unsigned short -- Check s

[PATCH] D38277: [compiler-rt}[CMake] Fix configuration on PowerPC with sanitizers

2017-09-27 Thread Aleksey Shlyapnikov via Phabricator via cfe-commits
alekseyshl added a comment. I wonder what's the error message? Comment at: cmake/base-config-ix.cmake:153 + # will fail with this option when building with a sanitizer. + set(OLD_CMAKE_REQUIRED_FLAGS ${CMAKE_REQUIRED_FLAGS}) + string(REPLACE "-nodefaultlibs" "" C

[PATCH] D38277: [compiler-rt}[CMake] Fix configuration on PowerPC with sanitizers

2017-09-26 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld created this revision. Herald added subscribers: mgorny, dberris, nemanjai. TEST_BIG_ENDIAN() performs compile tests that will fail with -nodefaultlibs when building under LLVM_USE_SANITIZER. https://reviews.llvm.org/D38277 Files: cmake/base-config-ix.cmake Index: cmake/base-config