diff options
author | 2015-12-11 11:17:49 +0000 | |
---|---|---|
committer | 2015-12-11 17:08:37 +0000 | |
commit | 6089e5be0d8f84ea3c96b5f46bf2387491fe7178 (patch) | |
tree | a9353f0e6e040aecb33d37cf9d8f1d47128d4eb5 /src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java | |
parent | cb8f278f42f3c8c7c69314c8cf68175718298216 (diff) |
Clean up the configuration-related code paths a bit.
- If we have a class object available, we can use .cast(Object)
- Only store the required options objects in CppConfigurationParameters
rather than keeping the entire BuildOptions
--
MOS_MIGRATED_REVID=109981236
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java index 63ff4ff41e..7ae7beb171 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java @@ -359,7 +359,7 @@ public class CppConfiguration extends BuildConfiguration.Fragment { protected CppConfiguration(CppConfigurationParameters params) throws InvalidConfigurationException { CrosstoolConfig.CToolchain toolchain = params.toolchain; - cppOptions = params.buildOptions.get(CppOptions.class); + cppOptions = params.cppOptions; this.hostSystemName = toolchain.getHostSystemName(); this.compiler = toolchain.getCompiler(); this.targetCpu = toolchain.getTargetCpu(); @@ -368,8 +368,7 @@ public class CppConfiguration extends BuildConfiguration.Fragment { this.targetLibc = toolchain.getTargetLibc(); this.crosstoolTop = params.crosstoolTop; this.ccToolchainLabel = params.ccToolchainLabel; - this.compilationMode = - params.buildOptions.get(BuildConfiguration.Options.class).compilationMode; + this.compilationMode = params.commonOptions.compilationMode; this.lipoContextCollector = cppOptions.lipoCollector; this.execRoot = params.execRoot; @@ -411,7 +410,7 @@ public class CppConfiguration extends BuildConfiguration.Fragment { } this.fdoSupport = new FdoSupport( - params.buildOptions.get(CppOptions.class).fdoInstrument, params.fdoZip, + cppOptions.fdoInstrument, params.fdoZip, cppOptions.lipoMode, execRoot); this.stripBinaries = |