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

Unified Diff: media/base/BUILD.gn

Issue 1152053004: Remove a few _sse2 targets now that we build with -msse2 everywhere. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: drunk Created 5 years, 6 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 | « cc/cc.gyp ('k') | media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/BUILD.gn
diff --git a/media/base/BUILD.gn b/media/base/BUILD.gn
index d81171eeb2a06f877793d523634a01b63d4dc367..29547f3da44e821e9691e106b314d21129af5369 100644
--- a/media/base/BUILD.gn
+++ b/media/base/BUILD.gn
@@ -272,11 +272,13 @@ source_set("base") {
}
if (current_cpu == "x86" || current_cpu == "x64") {
- sources += [ "simd/convert_yuv_to_rgb_x86.cc" ]
- deps += [
- ":media_yasm",
- ":media_sse2",
+ sources += [
+ "simd/convert_rgb_to_yuv_sse2.cc",
+ "simd/convert_rgb_to_yuv_ssse3.cc",
+ "simd/convert_yuv_to_rgb_x86.cc",
+ "simd/filter_yuv_sse2.cc",
]
+ deps += [ ":media_yasm" ]
}
configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
@@ -464,21 +466,6 @@ source_set("perftests") {
}
if (current_cpu == "x86" || current_cpu == "x64") {
- source_set("media_sse2") {
- sources = [
- "simd/convert_rgb_to_yuv_sse2.cc",
- "simd/convert_rgb_to_yuv_ssse3.cc",
- "simd/filter_yuv_sse2.cc",
- ]
- configs += [
- "//media:media_config",
- "//media:media_implementation",
- ]
- if (!is_win) {
- cflags = [ "-msse2" ]
- }
- }
-
import("//third_party/yasm/yasm_assemble.gni")
yasm_assemble("media_yasm") {
sources = [
« no previous file with comments | « cc/cc.gyp ('k') | media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698