Saturated arithmetic changed to use templates to fix build-issues

Source/wtf/SaturatedArithmetic.h

Index: Source/wtf/SaturatedArithmetic.h
diff --git a/Source/wtf/SaturatedArithmetic.h b/Source/wtf/SaturatedArithmetic.h
index ff4ecb3619ec9c15f0def09274a2eaa29134dc21..9cc561e8a8c7511721e9935ca81511b8ba2b5f76 100644
--- a/Source/wtf/SaturatedArithmetic.h
+++ b/Source/wtf/SaturatedArithmetic.h
@@ -86,13 +86,18 @@ inline int getMinSaturatedSetResultForTesting(int FractionalShift)
     return std::numeric_limits<int>::min();
 }
 
-ALWAYS_INLINE int saturatedSet(int value, int FractionalShift)
+template<size_t saturate, size_t fractionalShift>

[Sami, 2014/07/09 13:59:26]

Could you not calculate |saturate| from |fractionalShift| or do both need to be
separate constants that come from the expansion site?

[picksi, 2014/07/09 14:27:52]

In the previous version I did exactly this, but with android_full_debug=1 in
chromium.gyp_env it refused to work. I could only get it to compile when both
parameters came into the function as templated-parameters.

The "saturate" parameter is not actually needed by the C++ version and is only
here to create a common interface for the C++ and ARM version, so the calling
code can remain implementation agnostic.

+ALWAYS_INLINE int saturatedSetSigned(int value)
 {
+    // This function is implemented as a template to make a common interface
+    // for this and the ARM version, which will only compile under all compilers
+    // if it is implemented as a template specialized with constants.
+
     const int intMaxForLayoutUnit =
-        std::numeric_limits<int>::max() >> FractionalShift;
+        std::numeric_limits<int>::max() >> fractionalShift;
 
     const int intMinForLayoutUnit =
-        std::numeric_limits<int>::min() >> FractionalShift;
+        std::numeric_limits<int>::min() >> fractionalShift;
 
     if (value > intMaxForLayoutUnit)
         return std::numeric_limits<int>::max();
@@ -100,19 +105,23 @@ ALWAYS_INLINE int saturatedSet(int value, int FractionalShift)
     if (value < intMinForLayoutUnit)
         return std::numeric_limits<int>::min();
 
-    return value << FractionalShift;
+    return value << fractionalShift;
 }
 
-
-ALWAYS_INLINE int saturatedSet(unsigned value, int FractionalShift)
+template<size_t saturate, size_t fractionalShift>
+ALWAYS_INLINE int saturatedSetUnsigned(unsigned value)
 {
+    // This function is implemented as a template to make a common interface
+    // for this and the ARM version, which will only compile under all compilers
+    // if it is implemented as a template specialized with constants.
+
     const unsigned intMaxForLayoutUnit =
-        std::numeric_limits<int>::max() >> FractionalShift;
+        std::numeric_limits<int>::max() >> fractionalShift;
 
     if (value >= intMaxForLayoutUnit)
         return std::numeric_limits<int>::max();
 
-    return value << FractionalShift;
+    return value << fractionalShift;
 }
 
 #endif // CPU(ARM) && COMPILER(GCC)