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

Unified Diff: openssl.gyp

Issue 26651005: Use C source for openssl RC4 calculation instead of ASM (Closed) Base URL: https://chromium.googlesource.com/chromium/deps/openssl.git@master
Patch Set: Created 7 years, 2 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: openssl.gyp
diff --git a/openssl.gyp b/openssl.gyp
index 80733269d8bd30fb1aed695efa697d3a9e3aba8d..1bee4e3a50aa69ef3bd355e6a83f3420fd935f0d 100644
--- a/openssl.gyp
+++ b/openssl.gyp
@@ -65,6 +65,19 @@
['target_arch == "x64"', {
'sources': [ '<@(openssl_x86_64_sources)' ],
'sources!': [ '<@(openssl_x86_64_source_excludes)' ],
+ 'conditions': [
+ ['OS != "android"', {
+ # Because rc4-x86_64.S has a problem,
+ # We use the C rc4 source instead of the ASM source.
+ # This hurts performance, but it's not a problem
+ # because no production code uses openssl on x86-64.
+ 'sources/': [
+ ['exclude', 'openssl/crypto/rc4/asm/rc4-x86_64\\.S' ],
+ ['include', 'openssl/crypto/rc4/rc4_enc\\.c' ],
+ ['include', 'openssl/crypto/rc4/rc4_skey\\.c' ],
+ ],
+ }]
+ ],
'defines': [ '<@(openssl_x86_64_defines)' ],
'defines!': [ 'OPENSSL_NO_ASM' ],
'variables': {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698