Commit a0bd8ef0 by Jamie Madill

Use ANGLE_TRANSLATOR_STATIC to enforce a static build of the translator in chromium.

This prevents strange warnings from defining COMPONENT_BUILD=0. Also rename COMPILER_IMPLEMENTATION to ANGLE_TRANSLATOR_IMPLEMENTATION for better consistency in chromium. ANGLEBUG=477 R=zmo@chromium.org BUG= Review URL: https://codereview.appspot.com/14294046
parent 384c0cd5
......@@ -6,20 +6,20 @@
#ifndef _COMPILER_INTERFACE_INCLUDED_
#define _COMPILER_INTERFACE_INCLUDED_
#if (COMPONENT_BUILD)
#if defined(COMPONENT_BUILD) && !defined(ANGLE_TRANSLATOR_STATIC)
#if defined(_WIN32) || defined(_WIN64)
#if defined(COMPILER_IMPLEMENTATION)
#if defined(ANGLE_TRANSLATOR_IMPLEMENTATION)
#define COMPILER_EXPORT __declspec(dllexport)
#else
#define COMPILER_EXPORT __declspec(dllimport)
#endif // defined(COMPILER_IMPLEMENTATION)
#endif // defined(ANGLE_TRANSLATOR_IMPLEMENTATION)
#else // defined(WIN32)
#else // defined(_WIN32) || defined(_WIN64)
#define COMPILER_EXPORT __attribute__((visibility("default")))
#endif
#else // defined(COMPONENT_BUILD)
#else // defined(COMPONENT_BUILD) && !defined(ANGLE_TRANSLATOR_STATIC)
#define COMPILER_EXPORT
#endif
......
......@@ -168,7 +168,7 @@
'../include',
],
'defines': [
'COMPILER_IMPLEMENTATION',
'ANGLE_TRANSLATOR_IMPLEMENTATION',
],
'sources': ['<@(translator_sources)'],
# TODO(jschuh): http://crbug.com/167187 size_t -> int
......@@ -192,7 +192,7 @@
'../include',
],
'defines': [
'COMPILER_IMPLEMENTATION',
'ANGLE_TRANSLATOR_IMPLEMENTATION',
],
'sources': ['<@(translator_sources)'],
# TODO(jschuh): http://crbug.com/167187 size_t -> int
......@@ -214,8 +214,7 @@
'../include',
],
'defines': [
'COMPILER_IMPLEMENTATION',
'COMPONENT_BUILD=0',
'ANGLE_TRANSLATOR_STATIC',
],
'sources': ['<@(translator_sources)'],
# TODO(jschuh): http://crbug.com/167187 size_t -> int
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment