p96705831
/
platform_build
forked from openkylin/platform_build
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
62,976 Commits 1 Branch 0 Tags 68 MiB
Commit Graph

11 Commits

Author SHA1 Message Date
Kousik Kumar 405847ac19 Remove env-var-allowlist 
We no longer need to set PWD variable with the introduction of
`-fdebug-compilation-dir`, so removing env var allowlist of PWD variable
to RBE.

Change-Id: Ifa0592af519b6cc4364ee653f1a2174fd680bac1
 2020-10-15 15:53:13 -04:00
Kousik Kumar edb723a3f8 Rename wlist to allowlist 
The new release of re-client renames env_var_wlist to env_var_allowlist.
We need to make this change as part of the topic so that with the new
release we are able to deprecate this variable.

Bug: b/167677318
Change-Id: I988381f4f72061f7c6eb99d58593f92cfd618b1b
 2020-09-14 11:10:20 +00:00
Ramy Medhat ba6d777f25 Set RBE_DIR default value to point to prebuilts in tree. 
The new default points to prebuilts/remoteexecution-client/live.

Also sets the default exec strategy of Javac/R8/D8 to remote local
fallback.

Test: simple one action build
Change-Id: I3392a0f7ac6de818d0c5ba2e113fcb035c6fa0ae
 2020-08-13 11:49:30 -04:00
Kousik Kumar 151dd8fadb Remove double quotes in rbe.mk file 
These result in weird command line invocations when used with RBE.

Test: Ran an RBE build locally and made sure it still worked as expected
after the change.
Bug: b/163618332

Change-Id: Ic8a8adb3fffb357754384ddbf13684d944a9767e
 2020-08-12 13:38:59 +00:00
Kousik Kumar b0f055db13 Add RBE_CXX_COMPARE variable to locally disable compare mode for C++ 
When we do 64-32 migration, we want compare mode to be present only for
non-c++ actions as it would impact performance on 32-core machines
otherwise.

Bug: b/157414300
Test: Ran a build with "RBE_CXX_LINKS="true" RBE_CXX_LINKS_EXEC_STRATEGY="remote" RBE_CXX_COMPARE="false" RBE_compare="true" use_rbe mmma bionic"
and observed that local-executions happened only for link actions and
not for C++ actions.

Change-Id: Ia972c74d804085196a7b6ec755af63042c6b3909
 2020-05-26 11:57:02 -07:00
Kousik Kumar 3fd528e215 Update configuration for RemoteExecution actions 
1. Specify addition inputs needed for R8 / D8 / Javac actions so that
they can be successfully executed remotely.
2. Cleanup platform configuration so that pool-names are turned into a
common variable and re-used between javac / r8 / d8 actions.

Bug: b/152273259
Change-Id: I0231a96f6e4e094e98d0f89728f6ac3e34f78118
Merged-In: I0231a96f6e4e094e98d0f89728f6ac3e34f78118
 2020-04-02 12:07:47 -07:00
Kousik Kumar d463007dfc Add exec_strategy as CLI flags to all rewrapper commands 
Bug: b/148798195
Test: The same change is live in master-build-rbe-testing branch where
its not causing any behavior change, and also it works for
RemoteExecution where RBE_CXX_EXEC_STRATEGY variable is set.
 2020-02-05 16:06:47 +00:00
Ramy Medhat a9cc25d785 Add RBE support for Javac, R8, and D8 if respective env vars are set. 
Test: aosp_crosshatch build with and without the new variables.
Change-Id: I252ff5dd8e372edf8c86e25348c8a2bfab41d70d
 2020-01-28 10:25:25 -05:00
Ramy Medhat e8de0931a1 Add PWD to environment variable whitelist for RBE builds. 
Test: Build aosp with USE_RBE=1.

Change-Id: I4b51fbde330c6129654aa74f8a58dccf62cfa11d
 2019-08-27 23:50:31 +00:00
Ramy Medhat e5839f912b Add labels to rewrapper cc compile action. Prevent ccwrapper from being 
used in links.

Test: Built aosp-arm64_eng with and without USE_RBE=1

Change-Id: I8d04069219a9467c3de9a028eb68e3115173f1e2
 2019-08-12 14:13:31 -04:00
Ramy Medhat ec8b8dfa3e Add RBE mk file to support bootstrapping RBE and setting relevant 
environment variables.

Test: Built aosp_arm-user with and without USE_RBE. With USE_RBE,
used a simple proxy script in place of rewrapper.

Change-Id: Ifb6bfd9f7292346f4750a6a409aa0def4e3c0363
 2019-07-17 14:22:00 +00:00