Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(312)

Unified Diff: third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h

Issue 1611343002: wtf reformat test Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: pydent Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/wtf/WeakPtr.h ('k') | third_party/WebKit/Source/wtf/build_config.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h
diff --git a/third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h b/third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h
index d0a728015638e76d76491474496a007d7a5982ae..ab9415b9f25ffdd54ccab258b7b1d3cc3c88050e 100644
--- a/third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h
+++ b/third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h
@@ -9,99 +9,88 @@
#include <limits>
#include <stdint.h>
-ALWAYS_INLINE int32_t saturatedAddition(int32_t a, int32_t b)
-{
- int32_t result;
+ALWAYS_INLINE int32_t saturatedAddition(int32_t a, int32_t b) {
+ int32_t result;
- asm("qadd %[output],%[first],%[second]"
- : [output] "=r" (result)
- : [first] "r" (a),
- [second] "r" (b));
+ asm("qadd %[output],%[first],%[second]"
+ : [output] "=r"(result)
+ : [first] "r"(a), [second] "r"(b));
- return result;
+ return result;
}
-ALWAYS_INLINE int32_t saturatedSubtraction(int32_t a, int32_t b)
-{
- int32_t result;
+ALWAYS_INLINE int32_t saturatedSubtraction(int32_t a, int32_t b) {
+ int32_t result;
- asm("qsub %[output],%[first],%[second]"
- : [output] "=r" (result)
- : [first] "r" (a),
- [second] "r" (b));
+ asm("qsub %[output],%[first],%[second]"
+ : [output] "=r"(result)
+ : [first] "r"(a), [second] "r"(b));
- return result;
+ return result;
}
-inline int getMaxSaturatedSetResultForTesting(int FractionalShift)
-{
- // For ARM Asm version the set function maxes out to the biggest
- // possible integer part with the fractional part zero'd out.
- // e.g. 0x7fffffc0.
- return std::numeric_limits<int>::max() & ~((1 << FractionalShift)-1);
+inline int getMaxSaturatedSetResultForTesting(int FractionalShift) {
+ // For ARM Asm version the set function maxes out to the biggest
+ // possible integer part with the fractional part zero'd out.
+ // e.g. 0x7fffffc0.
+ return std::numeric_limits<int>::max() & ~((1 << FractionalShift) - 1);
}
-inline int getMinSaturatedSetResultForTesting(int FractionalShift)
-{
- return std::numeric_limits<int>::min();
+inline int getMinSaturatedSetResultForTesting(int FractionalShift) {
+ return std::numeric_limits<int>::min();
}
template <int FractionalShift>
-ALWAYS_INLINE int saturatedSet(int value)
-{
- // Figure out how many bits are left for storing the integer part of
- // the fixed point number, and saturate our input to that
- enum { Saturate = 32 - FractionalShift };
-
- int result;
-
- // The following ARM code will Saturate the passed value to the number of
- // bits used for the whole part of the fixed point representation, then
- // shift it up into place. This will result in the low <FractionShift> bits
- // all being 0's. When the value saturates this gives a different result
- // to from the C++ case; in the C++ code a saturated value has all the low
- // bits set to 1 (for a +ve number at least). This cannot be done rapidly
- // in ARM ... we live with the difference, for the sake of speed.
-
- asm("ssat %[output],%[saturate],%[value]\n\t"
- "lsl %[output],%[shift]"
- : [output] "=r" (result)
- : [value] "r" (value),
- [saturate] "n" (Saturate),
- [shift] "n" (FractionalShift));
-
- return result;
+ALWAYS_INLINE int saturatedSet(int value) {
+ // Figure out how many bits are left for storing the integer part of
+ // the fixed point number, and saturate our input to that
+ enum { Saturate = 32 - FractionalShift };
+
+ int result;
+
+ // The following ARM code will Saturate the passed value to the number of
+ // bits used for the whole part of the fixed point representation, then
+ // shift it up into place. This will result in the low <FractionShift> bits
+ // all being 0's. When the value saturates this gives a different result
+ // to from the C++ case; in the C++ code a saturated value has all the low
+ // bits set to 1 (for a +ve number at least). This cannot be done rapidly
+ // in ARM ... we live with the difference, for the sake of speed.
+
+ asm("ssat %[output],%[saturate],%[value]\n\t"
+ "lsl %[output],%[shift]"
+ : [output] "=r"(result)
+ : [value] "r"(value), [saturate] "n"(Saturate),
+ [shift] "n"(FractionalShift));
+
+ return result;
}
-
template <int FractionalShift>
-ALWAYS_INLINE int saturatedSet(unsigned value)
-{
- // Here we are being passed an unsigned value to saturate,
- // even though the result is returned as a signed integer. The ARM
- // instruction for unsigned saturation therefore needs to be given one
- // less bit (i.e. the sign bit) for the saturation to work correctly; hence
- // the '31' below.
- enum { Saturate = 31 - FractionalShift };
-
- // The following ARM code will Saturate the passed value to the number of
- // bits used for the whole part of the fixed point representation, then
- // shift it up into place. This will result in the low <FractionShift> bits
- // all being 0's. When the value saturates this gives a different result
- // to from the C++ case; in the C++ code a saturated value has all the low
- // bits set to 1. This cannot be done rapidly in ARM, so we live with the
- // difference, for the sake of speed.
-
- int result;
-
- asm("usat %[output],%[saturate],%[value]\n\t"
- "lsl %[output],%[shift]"
- : [output] "=r" (result)
- : [value] "r" (value),
- [saturate] "n" (Saturate),
- [shift] "n" (FractionalShift));
-
- return result;
+ALWAYS_INLINE int saturatedSet(unsigned value) {
+ // Here we are being passed an unsigned value to saturate,
+ // even though the result is returned as a signed integer. The ARM
+ // instruction for unsigned saturation therefore needs to be given one
+ // less bit (i.e. the sign bit) for the saturation to work correctly; hence
+ // the '31' below.
+ enum { Saturate = 31 - FractionalShift };
+
+ // The following ARM code will Saturate the passed value to the number of
+ // bits used for the whole part of the fixed point representation, then
+ // shift it up into place. This will result in the low <FractionShift> bits
+ // all being 0's. When the value saturates this gives a different result
+ // to from the C++ case; in the C++ code a saturated value has all the low
+ // bits set to 1. This cannot be done rapidly in ARM, so we live with the
+ // difference, for the sake of speed.
+
+ int result;
+
+ asm("usat %[output],%[saturate],%[value]\n\t"
+ "lsl %[output],%[shift]"
+ : [output] "=r"(result)
+ : [value] "r"(value), [saturate] "n"(Saturate),
+ [shift] "n"(FractionalShift));
+
+ return result;
}
-#endif // SaturatedArithmeticARM_h
+#endif // SaturatedArithmeticARM_h
« no previous file with comments | « third_party/WebKit/Source/wtf/WeakPtr.h ('k') | third_party/WebKit/Source/wtf/build_config.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698