Page MenuHomePhabricator

Revert [compiler-rt][CMake] Enable TF intrinsics on powerpc32 Linux
ClosedPublic

Authored by nikic on Aug 2 2022, 6:55 AM.

Details

Summary

As mentioned in https://reviews.llvm.org/D121379#3690593, this change broke the build of compiler-rt targeting powerpc using GCC. The 32-bit powerpc target is not supposed to emit 128-bit libcalls -- if it does, then that's a backend bug and needs to be fixed there.

This reverts commit 8f24a56a3a9363f353c8da318d97491a6818781d.

Diff Detail

Unit TestsFailed

TimeTest
2,100 msx64 debian > SanitizerCommon-asan-x86_64-Linux.SanitizerCommon-asan-x86_64-Linux::sanitizer_coverage_allowlist_ignorelist.cpp
Script: -- : 'RUN: at line 10'; DIR=/var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/test/sanitizer_common/asan-x86_64-Linux/Output/sanitizer_coverage_allowlist_ignorelist.cpp.tmp_workdir
2,020 msx64 debian > SanitizerCommon-lsan-x86_64-Linux.SanitizerCommon-lsan-x86_64-Linux::sanitizer_coverage_allowlist_ignorelist.cpp
Script: -- : 'RUN: at line 10'; DIR=/var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/test/sanitizer_common/lsan-x86_64-Linux/Output/sanitizer_coverage_allowlist_ignorelist.cpp.tmp_workdir
2,100 msx64 debian > SanitizerCommon-msan-x86_64-Linux.SanitizerCommon-msan-x86_64-Linux::sanitizer_coverage_allowlist_ignorelist.cpp
Script: -- : 'RUN: at line 10'; DIR=/var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/test/sanitizer_common/msan-x86_64-Linux/Output/sanitizer_coverage_allowlist_ignorelist.cpp.tmp_workdir

Event Timeline

nikic created this revision.Aug 2 2022, 6:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 2 2022, 6:55 AM
nikic requested review of this revision.Aug 2 2022, 6:55 AM
glaubitz accepted this revision.Aug 5 2022, 1:42 AM
This revision is now accepted and ready to land.Aug 5 2022, 1:42 AM
This revision was landed with ongoing or failed builds.Aug 5 2022, 7:43 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 5 2022, 7:43 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript