https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=293845

--- Comment #4 from Mark Millard <[email protected]> ---
(In reply to Mark Millard from comment #3)

Well, there is a patch too:

--- pxr/base/arch/timing.h.orig 2025-10-24 16:21:56 UTC
+++ pxr/base/arch/timing.h
@@ -19,7 +19,7 @@
 /// \addtogroup group_arch_SystemFunctions
 ///@{

-#if defined(ARCH_OS_LINUX) && defined(ARCH_CPU_INTEL)
+#if defined(ARCH_OS_LINUX) || defined(ARCH_OS_FREEBSD) ||
defined(ARCH_OS_FREEBSD) && defined(ARCH_CPU_INTEL)
 #include <x86intrin.h>
 #elif defined(ARCH_OS_WASM_VM)
 #include <emscripten.h>

(The patch indicates that I was apparently looking at the
wrong vintage of the upstream file vs. what the patch is
expected to deal with.)

The patch is worse: the "|| defined(ARCH_OS_FREEBSD) ||" means that
all such contexts idetnifeid for FreeBSD will include x86intrin.h .
What follows does not matter.

My guess is that the intent was more like:

#if (defined(ARCH_OS_LINUX) || defined(ARCH_OS_FREEBSD)) &&
defined(ARCH_CPU_INTEL)

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to