Implemented runtime macro free code

Author: Przemog1

include/nbl/builtin/hlsl/subgroup/arithmetic_portability.hlsl

@@ -8,6 +8,7 @@
 #include "nbl/builtin/hlsl/subgroup/basic.hlsl"
 
 #include "nbl/builtin/hlsl/subgroup/arithmetic_portability_impl.hlsl"
+#include "nbl/builtin/hlsl/jit/device_capabilities.hlsl"
 
 
 namespace nbl
@@ -17,20 +18,16 @@ namespace hlsl
 namespace subgroup
 {
 
-#ifdef NBL_GLSL_LIMIT_SHADER_SUBGROUP_ARITHMETIC
-#define IMPL native
-#else
-#define IMPL portability
-#endif
+// TODO: change to alias using template when DXC finally fixes SPIR-V codegen impartity
+//template<class Binop, class device_capability_traits/*=TODO: Atil give us the traits so we can default with `void`!*/>
+//struct reduction : impl::reduction<Binop,device_capability_traits::subgroupArithmetic> {};
 
 template<class Binop>
-struct reduction : IMPL::reduction<Binop> {};
+struct reduction : impl::reduction<Binop,nbl::hlsl::jit::device_capabilities::subgroupArithmetic> {};
 template<class Binop>
-struct inclusive_scan : IMPL::inclusive_scan<Binop> {};
+struct inclusive_scan : impl::inclusive_scan<Binop,nbl::hlsl::jit::device_capabilities::subgroupArithmetic> {};
 template<class Binop>
-struct exclusive_scan : IMPL::exclusive_scan<Binop> {};
-
-#undef IMPL
+struct exclusive_scan : impl::exclusive_scan<Binop,nbl::hlsl::jit::device_capabilities::subgroupArithmetic> {};
 
 }
 }

include/nbl/builtin/hlsl/subgroup/arithmetic_portability_impl.hlsl

@@ -22,17 +22,34 @@ namespace hlsl
 namespace subgroup
 {
 
-namespace native
+namespace impl
 {
-
-template<class Binop, typename T=typename Binop::type_t>
+template<class Binop, bool native> // might need a 3rd default param `typename T=typename Binop::type_t`
 struct reduction;
-template<class Binop, typename T=typename Binop::type_t>
+template<class Binop, bool native>
 struct inclusive_scan;
-template<class Binop, typename T=typename Binop::type_t>
+template<class Binop, bool native>
 struct exclusive_scan;
 
-#define SPECIALIZE(NAME,BINOP,SUBGROUP_OP) template<typename T> struct NAME<BINOP<T>,T> \
+// native
+template<class Binop>
+struct reduction<Binop, true>;
+template<class Binop>
+struct inclusive_scan<Binop, true>;
+template<class Binop>
+struct exclusive_scan<Binop, true>;
+
+// portability
+template<class Binop>
+struct reduction<Binop, false>;
+template<class Binop>
+struct inclusive_scan<Binop, false>;
+template<class Binop>
+struct exclusive_scan<Binop, false>;
+
+// specialize native
+
+#define SPECIALIZE(NAME,BINOP,SUBGROUP_OP) template<typename T> struct NAME<BINOP<T>,true> \
 { \
     using type_t = T; \
  \
@@ -56,19 +73,16 @@ SPECIALIZE_ALL(maximum,Max);
 #undef SPECIALIZE_ALL
 #undef SPECIALIZE
 
-}
+// specialize portability
 
-namespace portability
-{
-    
 // WARNING
 // THIS PORTABILITY IMPLEMENTATION USES SHUFFLE OPS
 // Shuffles where you attempt to read an invactive lane, return garbage, 
 // which means that our portability reductions and prefix sums will also return garbage/UB/UV
 // Always use the native subgroup_arithmetic extensions if supported
 
 template<class Binop>
-struct inclusive_scan
+struct inclusive_scan<Binop, false>
 {
     using type_t = typename Binop::type_t;
 
@@ -97,13 +111,13 @@ struct inclusive_scan
 };
 
 template<class Binop>
-struct exclusive_scan
+struct exclusive_scan<Binop, false>
 {
     using type_t = typename Binop::type_t;
 
     type_t operator()(type_t value)
     {
-        value = inclusive_scan<Binop>::__call(value);
+        value = inclusive_scan<Binop,false>::__call(value);
         // can't risk getting short-circuited, need to store to a var
         type_t left = glsl::subgroupShuffleUp<type_t>(value,1);
         // the first invocation doesn't have anything in its left so we set to the binop's identity value for exlusive scan
@@ -112,16 +126,17 @@ struct exclusive_scan
 };
 
 template<class Binop>
-struct reduction
+struct reduction<Binop, false>
 {
     using type_t = typename Binop::type_t;
 
     type_t operator()(NBL_CONST_REF_ARG(type_t) value)
     {
         // take the last subgroup invocation's value for the reduction
-        return BroadcastLast<type_t>(inclusive_scan<Binop>::__call(value));
+        return BroadcastLast<type_t>(inclusive_scan<Binop,false>::__call(value));
     }
 };
+
 }
 
 }