remove SK_SUPPORT_LEGACY_DIVBITS_UB -- chrome no longer defines this
authorreed <reed@google.com>
Thu, 28 Apr 2016 01:45:36 +0000 (18:45 -0700)
committerCommit bot <commit-bot@chromium.org>
Thu, 28 Apr 2016 01:45:36 +0000 (18:45 -0700)
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1927823002

TBR=mtklein@google.com

Review-Url: https://codereview.chromium.org/1927823002

include/private/SkFixed.h

index c90d5e9..dade239 100644 (file)
@@ -98,14 +98,9 @@ static inline SkFixed SkDoublePinToFixed(double x) {
 #define SkFixedAbs(x)       SkAbs32(x)
 #define SkFixedAve(a, b)    (((a) + (b)) >> 1)
 
-// Blink layout tests are baselined to Clang optimizing through undefined behavior in SkDivBits.
-#if defined(SK_SUPPORT_LEGACY_DIVBITS_UB)
-    #define SkFixedDiv(numer, denom) SkDivBits(numer, denom, 16)
-#else
-    // The divide may exceed 32 bits. Clamp to a signed 32 bit result.
-    #define SkFixedDiv(numer, denom) \
-        SkToS32(SkTPin<int64_t>((SkLeftShift((int64_t)numer, 16) / denom), SK_MinS32, SK_MaxS32))
-#endif
+// The divide may exceed 32 bits. Clamp to a signed 32 bit result.
+#define SkFixedDiv(numer, denom) \
+    SkToS32(SkTPin<int64_t>((SkLeftShift((int64_t)numer, 16) / denom), SK_MinS32, SK_MaxS32))
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////
 // Now look for ASM overrides for our portable versions (should consider putting this in its own file)