This is an archive of the discontinued LLVM Phabricator instance.

compiler-rt: use correct builtins library when linking sanitizers
Needs ReviewPublic

Authored by Keruspe on Nov 14 2019, 3:48 AM.

Details

Reviewers
beanz
Summary

When building on x86_64 with -DCOMPILER_RT_USE_BUILTINS_LIBRARY=ON we try to build sanitizers for both x86_64 and i386 but we tried to link both to the x86_64 builtins.
This patch uses the correct builtins to link into the sanitizers.

Based on patch attached here: https://bugs.llvm.org/show_bug.cgi?id=38025

Diff Detail

Event Timeline

Keruspe created this revision.Nov 14 2019, 3:48 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptNov 14 2019, 3:48 AM
Herald added subscribers: llvm-commits, Restricted Project, mgorny, dberris. · View Herald Transcript
Keruspe updated this revision to Diff 229266.Nov 14 2019, 3:52 AM

Fix variable indirection. (Wrong verison of the patch was attached)