Try to make it build on clang

eggrobin · Dec 30, 2024 · 4c24059 · 4c24059
1 parent 4e38e35
commit 4c24059
Show file tree

Hide file tree

Showing 3 changed files with 38 additions and 5 deletions.
diff --git a/nanobenchmarks/function_registry.hpp b/nanobenchmarks/function_registry.hpp
@@ -5,8 +5,6 @@
 #include <string>
 #include <string_view>
 
-#include "base/macros.hpp"
-
 namespace principia {
 namespace nanobenchmarks {
 namespace _function_registry {

diff --git a/nanobenchmarks/microarchitectures.cpp b/nanobenchmarks/microarchitectures.cpp
@@ -6,13 +6,36 @@
 #include <vector>
 
 #include "base/cpuid.hpp"
+#include "base/macros.hpp"  // 🧙 For PRINCIPIA_COMPILER_CLANG.
 #include "glog/logging.h"
 
 BENCHMARK_EXTERN_C_FUNCTION(identity);
 BENCHMARK_EXTERN_C_FUNCTION(sqrtps_xmm0_xmm0);
 BENCHMARK_EXTERN_C_FUNCTION(sqrtsd_xmm0_xmm0);
 BENCHMARK_EXTERN_C_FUNCTION(mulsd_xmm0_xmm0);
 BENCHMARK_EXTERN_C_FUNCTION(mulsd_xmm0_xmm0_4x);
+#if PRINCIPIA_COMPILER_CLANG
+asm(R"(
+.intel_syntax
+identity:
+  ret
+sqrtps_xmm0_xmm0:
+  sqrtps xmm0, xmm0
+  ret
+sqrtsd_xmm0_xmm0:
+  sqrtsd xmm0, xmm0
+  ret
+mulsd_xmm0_xmm0:
+  mulsd xmm0, xmm0
+  ret
+mulsd_xmm0_xmm0_4x:
+  mulsd xmm0, xmm0
+  mulsd xmm0, xmm0
+  mulsd xmm0, xmm0
+  mulsd xmm0, xmm0
+  ret
+)");
+#endif
 
 namespace principia {
 namespace nanobenchmarks {

diff --git a/nanobenchmarks/performance_settings_controller.cpp b/nanobenchmarks/performance_settings_controller.cpp
@@ -6,13 +6,15 @@
 #include <utility>
 #include <string_view>
 
+#include "absl/flags/flag.h"
+#include "base/macros.hpp"  // 🧙 For OS_WIN.
+#include "glog/logging.h"
+
+#if OS_WIN
 #include <windows.h>
 #include <powersetting.h>
 #include <powrprof.h>
 
-#include "absl/flags/flag.h"
-#include "glog/logging.h"
-
 ABSL_FLAG(bool,
           keep_perf_boost,
           false,
@@ -23,12 +25,14 @@ ABSL_FLAG(bool,
           keep_throttling,
           false,
           "Whether to allow processor throttling during benchmark execution");
+#endif
 
 namespace principia {
 namespace nanobenchmarks {
 namespace _performance_settings_controller {
 namespace internal {
 
+#if OS_WIN
 class WindowsPerformanceSettingsController
     : public PerformanceSettingsController {
  public:
@@ -295,6 +299,14 @@ WindowsPerformanceSettingsController::ReadAndPrintAllowThrottlingACDC() const {
       "PROCESSOR_THROTTLE DC={} ({})", dc, ProcessorThrottleToString(dc));
   return {ac, dc};
 }
+#else
+
+not_null<std::unique_ptr<PerformanceSettingsController>>
+PerformanceSettingsController::New() {
+  return make_not_null_unique<PerformanceSettingsController>();
+}
+
+#endif
 
 }  // namespace internal
 }  // namespace _performance_settings_controller