From e695e33e8709489e9c1f3331e39c53e0b587ba80 Mon Sep 17 00:00:00 2001
From: Alex Voicu <alexandru.voicu@amd.com>
Date: Wed, 20 Sep 2017 18:07:51 +0100
Subject: [PATCH] Mark global scope inline functions as static. Take arguments
 by value, and not by reference, since they are essentially a single
 std::uint64_t wrapped in a struct.

---
 hc2/headers/functions/hsa_interfaces.hpp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/hc2/headers/functions/hsa_interfaces.hpp b/hc2/headers/functions/hsa_interfaces.hpp
index bcc4b71081e..9eef5f6ee91 100644
--- a/hc2/headers/functions/hsa_interfaces.hpp
+++ b/hc2/headers/functions/hsa_interfaces.hpp
@@ -18,21 +18,23 @@
 #include <memory>
 #include <utility>
 
+static
 inline
-bool operator==(const hsa_agent_t& x, const hsa_agent_t& y)
+bool operator==(hsa_agent_t x, hsa_agent_t y)
 {
     return x.handle == y.handle;
 }
 
+static
 inline
-bool operator==(const hsa_isa_t& x, const hsa_isa_t& y)
+bool operator==(hsa_isa_t x, hsa_isa_t y)
 {
     return x.handle == y.handle;
 }
 
+static
 inline
-bool operator==(
-    const hsa_executable_symbol_t& x, const hsa_executable_symbol_t& y)
+bool operator==(hsa_executable_symbol_t x, hsa_executable_symbol_t y)
 {
     return x.handle == y.handle;
 }