Set the LLVM module's target triple

The target triple is used by MemorySanitizer to determine where to map the shadow and origin data, which is OS specific. When not set, the MemorySanitizer::initializeModule() method, which inserts an __msan_init call to the module's constructor list, produces an "unsupported operating system" error. Note that JITs like Reactor have no use for a module constructor list (__msan_init gets called during executable startup), but there is currently no way to skip this initialization. Leaving it unused is fine though. Likewise, setting the target triple has not other known JIT uses at the moment, but it can't hurt. Bug: b/155148722 Change-Id: Ia31d8c71ee495aadd91530a8d1c5fdc93add8e01 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/49853 Kokoro-Result: kokoro <noreply+kokoro@google.com> Tested-by: Nicolas Capens <nicolascapens@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>

Set the LLVM module's target triple
959f4198 · Nicolas Capens · Nicolas Capens · eaf1c535 · 959f4198
Commit 959f4198 authored Nov 02, 2020 by Nicolas Capens Committed by Nicolas Capens Nov 02, 2020
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 0 deletions

LLVMJIT.cpp src/Reactor/LLVMJIT.cpp +1 -0

No files found.
--- a/src/Reactor/LLVMJIT.cpp
+++ b/src/Reactor/LLVMJIT.cpp
@@ -597,6 +597,7 @@ JITBuilder::JITBuilder(const rr::Config &config)
    , module(new llvm::Module("", context))
    , builder(new llvm::IRBuilder<>(context))
 {
+	module->setTargetTriple(LLVM_DEFAULT_TARGET_TRIPLE);
 	module->setDataLayout(JITGlobals::get()->getDataLayout());
 }