Skip to content

Commit 69c9134

Browse files
gbaraldiclaude
andcommitted
processor: temporarily enable debug output + analyzegc fix
Enable cpufeatures debug output unconditionally to diagnose i686 CI failure. Also hide jl_get_llvm_target from clang analyzer. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent ee1cb4f commit 69c9134

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

deps/cpufeatures.version

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,5 @@
33
## source build
44
CPUFEATURES_VER := 0.2.0
55
CPUFEATURES_GIT_URL := https://github.com/gbaraldi/cpufeatures.git
6-
CPUFEATURES_TAR_URL := https://github.com/gbaraldi/cpufeatures/archive/2e40d39f94337875f69d0b6fafb440a4e13fe1fb.tar.gz
7-
CPUFEATURES_SHA := 2e40d39f94337875f69d0b6fafb440a4e13fe1fb
6+
CPUFEATURES_TAR_URL := https://github.com/gbaraldi/cpufeatures/archive/5319d0c.tar.gz
7+
CPUFEATURES_SHA := 5319d0c

src/processor.cpp

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -274,12 +274,8 @@ static_assert(TARGET_TABLES_LLVM_VERSION_MAJOR == LLVM_VERSION_MAJOR,
274274
// ============================================================================
275275

276276
static bool cpufeatures_debug_enabled() {
277-
static int enabled = -1;
278-
if (enabled == -1) {
279-
const char *debug_env = getenv("JULIA_DEBUG");
280-
enabled = debug_env && (strstr(debug_env, "cpufeatures") || strstr(debug_env, "all"));
281-
}
282-
return enabled;
277+
// Temporarily always enabled for CI debugging
278+
return true;
283279
}
284280

285281
#define CF_DEBUG(...) do { if (cpufeatures_debug_enabled()) jl_safe_printf(__VA_ARGS__); } while (0)
@@ -617,6 +613,7 @@ jl_image_t jl_init_processor_pkgimg(jl_image_buf_t image)
617613
return parse_sysimg(image, pkgimg_init_cb, NULL);
618614
}
619615

616+
#ifndef __clang_analyzer__
620617
std::pair<std::string, std::vector<std::string>>
621618
jl_get_llvm_target(const char *cpu_target, bool imaging, uint32_t &flags)
622619
{
@@ -639,6 +636,7 @@ jl_get_llvm_target(const char *cpu_target, bool imaging, uint32_t &flags)
639636

640637
return std::make_pair(spec.cpu_name, std::move(features));
641638
}
639+
#endif
642640

643641
#ifndef __clang_analyzer__
644642
const std::pair<std::string, std::string> &jl_get_llvm_disasm_target(void)

0 commit comments

Comments
 (0)