Skip to content

Commit

Permalink
Tentative Windows fix.
Browse files Browse the repository at this point in the history
  • Loading branch information
bluescarni committed Sep 2, 2024
1 parent cfdce12 commit db35939
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/llvm_state.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -309,17 +309,17 @@ llvm::orc::JITTargetMachineBuilder create_jit_tmb(unsigned opt_level, code_model

// LCOV_EXCL_START

#if LLVM_VERSION_MAJOR >= 17

// NOTE: the code model setup is working only on LLVM>=19 (or at least
// LLVM 18 + patches, as in the conda-forge LLVM package), due to this bug:
//
// https://github.com/llvm/llvm-project/issues/88115
//
// Additionally, there are indications from our CI that attempting to set
// the code model before LLVM 17 might just be buggy, as we see widespread
// the code model before LLVM 17 or on Windows might just be buggy, as we see widespread
// ASAN failures all over the place. Thus, let us not do anything with the code
// model setting before LLVM 17.
// model setting before LLVM 17 or on Windows.

#if LLVM_VERSION_MAJOR >= 17 && !defined(_WIN32)

// Setup the code model.
switch (c_model) {
Expand Down

0 comments on commit db35939

Please sign in to comment.