diff options
author | mtklein <mtklein@chromium.org> | 2015-12-08 10:53:01 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-12-08 10:53:01 -0800 |
commit | 1f6a1bd969db92daa5fa60517d6b8fc6eef8cb61 (patch) | |
tree | 7ba4adc6c8d4714054b49db44f531348d906250d /bench/MathBench.cpp | |
parent | cdd4682f450b98bad71b8cb0772c02e0902aaa0d (diff) |
spin off remaining integer overflow fixes
- Carmack rsqrt uses an int where it wants a uint32_t.
- turn off all santizers (including signed-integer-overflow) in third_party/externals/sftntly.
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-ASAN-Trybot
BUG=skia:4635
Review URL: https://codereview.chromium.org/1511643002
Diffstat (limited to 'bench/MathBench.cpp')
-rw-r--r-- | bench/MathBench.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bench/MathBench.cpp b/bench/MathBench.cpp index a5cc033c6f..fb35a8aafe 100644 --- a/bench/MathBench.cpp +++ b/bench/MathBench.cpp @@ -126,7 +126,7 @@ private: static inline float SkFastInvSqrt(float x) { float xhalf = 0.5f*x; - int i = *SkTCast<int*>(&x); + uint32_t i = *SkTCast<uint32_t*>(&x); i = 0x5f3759df - (i>>1); x = *SkTCast<float*>(&i); x = x*(1.5f-xhalf*x*x); |