Fix bug

FLAMEGPU · Nov 23, 2021 · 4d1e461 · 4d1e461
1 parent a5aa729
commit 4d1e461
Showing 1 changed file with 4 additions and 12 deletions.
diff --git a/include/flamegpu/runtime/utility/DeviceMacroProperty.cuh b/include/flamegpu/runtime/utility/DeviceMacroProperty.cuh
@@ -102,14 +102,12 @@ class DeviceMacroProperty : public ReadOnlyDeviceMacroProperty<T, I, J, K, W> {
      * atomic add
      * @param val The 2nd operand
      * @return (this + val)
-     * @note Only suitable where T is type int32_t, uint32_t, uint64_t, float, double
      */
     __device__ __forceinline__ T operator+(const T& val) const;
     /**
      * atomic subtraction
      * @param val The 2nd operand
      * @return (this - val)
-     * @note Only suitable where T is type int32_t or uint32_t
      */
     __device__ __forceinline__ T operator-(const T& val) const;
     /**
@@ -293,35 +291,29 @@ __device__ __forceinline__ DeviceMacroProperty<T, I, J, K, W>& DeviceMacroProper
 }
 template<typename T, unsigned int I, unsigned int J, unsigned int K, unsigned int W>
 __device__ __forceinline__ T DeviceMacroProperty<T, I, J, K, W>::operator+(const T& val) const {
-    static_assert(std::is_same<T, int32_t>::value ||
-        std::is_same<T, uint32_t>::value ||
-        std::is_same<T, uint64_t>::value ||
-        std::is_same<T, float>::value ||
-        std::is_same<T, double>::value, "atomic add only supports the types int32_t/uint32_t/uint64_t/float/double.");
 #if !defined(SEATBELTS) || SEATBELTS
     if (I != 1 || J != 1 || K != 1 || W != 1) {
         DTHROW("Indexing error, property has more dimensions.\n");
         return { };
     } else if (this->ptr == nullptr) {
         return { };
     }
-    this->setCheckWriteFlag();
+    this->setCheckReadFlag();
 #endif
-    return atomicAdd(this->ptr, val) + val;
+    return *this->ptr + val;
 }
 template<typename T, unsigned int I, unsigned int J, unsigned int K, unsigned int W>
 __device__ __forceinline__ T DeviceMacroProperty<T, I, J, K, W>::operator-(const T& val) const {
-    static_assert(std::is_same<T, uint32_t>::value || std::is_same<T, int32_t>::value, "atomic subtract only supports the types int32_t/uint32_t.");
 #if !defined(SEATBELTS) || SEATBELTS
     if (I != 1 || J != 1 || K != 1 || W != 1) {
         DTHROW("Indexing error, property has more dimensions.\n");
         return { };
     } else if (this->ptr == nullptr) {
         return { };
     }
-    this->setCheckWriteFlag();
+    this->setCheckReadFlag();
 #endif
-    return atomicSub(this->ptr, val) - val;
+    return *this->ptr - val;
 }
 template<typename T, unsigned int I, unsigned int J, unsigned int K, unsigned int W>
 __device__ __forceinline__ T DeviceMacroProperty<T, I, J, K, W>::operator++() {