diff options
author | Yun Peng <pcloudy@google.com> | 2016-10-06 10:55:54 +0000 |
---|---|---|
committer | Damien Martin-Guillerez <dmarting@google.com> | 2016-10-06 10:58:43 +0000 |
commit | ffdc05d2278d7f9c6e299c923019f689cde5fe76 (patch) | |
tree | 3f927eca381f68465fc0b142c9c499b18d503f62 /src/test/java | |
parent | 27085f44a168b502542cda6216863601e64da833 (diff) |
Add action_config and feature for linking on Windows
Also implemented whole archive feature on Windows
1. Pulled action_configs and features from CppLinkActionConfigs.java
2. Deleted many features not working on Windows. (AFAIK)
including:
symbol_counts
output_execpath_flags_executable
global_whole_archive_open
runtime_root_flags
global_whole_archive_close
force_pic_flags
3. Added c++-link-interface-dynamic-library action config
Not sure there is such thing on Windows, but it's currently in MANDATORY_LINK_TARGET_TYPES
4. Added build variable "whole_archive_object_files_params"
5. Automatically detect if linker supports /WHOLEARCHIVE and use the corresponding build variable
--
Change-Id: I232798a0eb1a3291f972b313a81e678b0121d58c
Reviewed-on: https://bazel-review.googlesource.com/#/c/6414
MOS_MIGRATED_REVID=135342093
Diffstat (limited to 'src/test/java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java b/src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java index 056fb3bb83..7e9a9dce85 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java @@ -160,4 +160,9 @@ public class LinkBuildVariablesTest extends BuildViewTestCase { assertThat(wholeArchiveInputVariableValue).isEmpty(); ; } + + /** + * TODO(pcloudy): Add test for testing that necessary build variables are populated + * when alwayslink=1. + */ } |