diff options
| author |  Googler <noreply@google.com> | 2017-03-14 13:29:10 +0000 |
|---|---|---|
| committer |  Yun Peng <pcloudy@google.com> | 2017-03-14 19:50:00 +0000 |
| commit | 278b58e23c6a7057e641f16be2700ef116680524 (patch) | |
| tree | edf1c2f992fe0fcb6ac120f3db20d84cca55415a /src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java | |
| parent | 95a4dd0269d95a2a5dc8a952573ef6d83791bbfb (diff) | |
Refactor scheduling of LTO Backend actions into CppLinkActionBuilder
This was suggested in the review for unknown commit, which adds ThinLTO support
to another client of CppLinkActionBuilder.
The change required changing the constructor to take a FdoSupportProvider
object instead of a FdoSupport object, so required changes to all callers.
--
PiperOrigin-RevId: 150060046
MOS_MIGRATED_REVID=150060046
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java b/src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java index 612d81d285..f87e24e8d8 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java +++ b/src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java @@ -184,8 +184,7 @@ public class CrosstoolCompilationSupport extends CompilationSupport { .addVariableCategory(VariableCategory.FULLY_LINK_VARIABLES) .build(); CppLinkAction fullyLinkAction = - new CppLinkActionBuilder( - ruleContext, outputArchive, ccToolchain, fdoSupport.getFdoSupport()) + new CppLinkActionBuilder(ruleContext, outputArchive, ccToolchain, fdoSupport) .addActionInputs(objcProvider.getObjcLibraries()) .addActionInputs(objcProvider.getCcLibraries()) .addActionInputs(objcProvider.get(IMPORTED_LIBRARY).toSet()) @@ -249,7 +248,7 @@ public class CrosstoolCompilationSupport extends CompilationSupport { CcToolchainProvider ccToolchain = CppHelper.getToolchain(ruleContext, ":cc_toolchain"); FdoSupportProvider fdoSupport = CppHelper.getFdoSupport(ruleContext, ":cc_toolchain"); CppLinkAction executableLinkAction = - new CppLinkActionBuilder(ruleContext, binaryToLink, ccToolchain, fdoSupport.getFdoSupport()) + new CppLinkActionBuilder(ruleContext, binaryToLink, ccToolchain, fdoSupport) .setMnemonic("ObjcLink") .addActionInputs(bazelBuiltLibraries) .addActionInputs(objcProvider.getCcLibraries()) |