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

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

Issue 1436153002: Apply clang-format with Chromium-style without column limit. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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/dtoa.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 5527cacf9b2a3f47988c7963d2d2101cc8353b9e..60b4b6fcafbb976aa6e6fcbb0258657fc20115ee 100644
--- a/third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h
+++ b/third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h
@@ -9,97 +9,92 @@
#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();
}
-ALWAYS_INLINE int saturatedSet(int value, int FractionalShift)
-{
- // Figure out how many bits are left for storing the integer part of
- // the fixed point number, and saturate our input to that
- const int 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, int FractionalShift) {
+ // Figure out how many bits are left for storing the integer part of
+ // the fixed point number, and saturate our input to that
+ const int 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(unsigned value, int FractionalShift)
-{
- // 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.
- const int 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, int FractionalShift) {
+ // 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.
+ const int 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/dtoa.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698