Skip to content

Commit

Permalink
remove -ffp-contract
Browse files Browse the repository at this point in the history
We don't set it in Chrome, so this'll make our local bots a little more
consistent with what we see in Chrome.  Very, very minor stuff.

Bug: skia:9103
Change-Id: Iecedd367780b6a721bc49dfa2d0a7837eb3bd28d
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/215144
Auto-Submit: Mike Klein <mtklein@google.com>
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
  • Loading branch information
Mike Klein authored and Skia Commit-Bot committed May 22, 2019
1 parent c4dbc37 commit f77dbd0
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -279,14 +279,6 @@ opts("hsw") {
} else {
cflags = [ "-march=haswell" ]
}

# Oddly, clang-cl doesn't recognize this as a valid flag.
# If it ever does, it'd nice to move this up with -mavx2 and co.
if (is_clang && !is_win) {
# This flag lets Clang generate FMAs when it sees a mul-then-add. It's optional,
# but nice to have, generating slightly better code for paths without explicit FMAs.
cflags += [ "-ffp-contract=fast" ]
}
}

# Any feature of Skia that requires third-party code should be optional and use this template.
Expand Down

0 comments on commit f77dbd0

Please sign in to comment.