Age | Commit message (Collapse) | Author |
|
ToT clang needs --prefix is needed to find the correct tools.
BUG=chromium:1182224
TEST=Updated compiler wrapper in SimpleChrome, nacl builds.
Change-Id: I8073e1a5482a23f38bb1e0d88e2b31ee2d97fa6e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2721199
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Our old kefka machines got moved out of the lab; they have given
us two new ones. This updates the default_remotes file to
contain the names of the new kefka machines.
BUG=b/180983943
TEST=None
Change-Id: I3972528f5454fe6993aa96a4c91d0a6ade4780ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2719301
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
|
|
There was a small formatting error in gcc_flags.go. This caused a minor
issue when running bundle.py since the result was not accepted as is by
the presubmit.
BUG=None
TEST=None
Change-Id: I47709e5fdfe1e0f8b7cbc731e70b734196bd0ccc
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2717925
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Disables Goma and CCahce when rusage monitoring is requested.
This allows for usable metrics to be produced for more packages.
BUG=chromium:1156314
TEST=Modified and ran unit tests
Change-Id: I36415b4df6ce033b23ff8055e900233886e1889d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2713228
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Denis Nikitin <denik@chromium.org>
|
|
cherrypick_cl.py fails whenever a commit is already present, but the
revert checker should simply note this and move on.
BUG=chromium:1181376
TEST=nightly_revert_checker_test.py
Change-Id: I04500fef98a95607df3cb23bf0a901d3ba5062a7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2716971
Auto-Submit: Christopher Di Bella <cjdb@google.com>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Update chromeos-kernel-3_18 to R90-13768.0-1613385365
Update chromeos-kernel-4_4 to R90-13768.0-1613385307
Update chromeos-kernel-4_14 to R90-13778.0-1613385190
Update chromeos-kernel-4_19 to R90-13772.0-1613385133
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I3d14df99dd8be75feb51d8d701a88e20127335af
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2707323
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
|
|
BUG=chromium:1085465
TEST=nightly_revert_checker_test.py
Change-Id: I8069bb3f6ca8d07f54568b85373910a192d11ea5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2680123
Auto-Submit: Christopher Di Bella <cjdb@google.com>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Recent update to gcc 10 supports up to tigerlake. This CL
removes depcreated march conversions
BUG=chromium:1167265
TEST=Verify unit tests pass
Change-Id: Ic20c9901963deb0e9985f4bcd57c7651cb8aeb3b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2691128
Commit-Queue: Michael Cheng <michael.cheng@intel.com>
Tested-by: Michael Cheng <michael.cheng@intel.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
GCC 10 does not support march=alderlake. This CL
maps alderlake to skylake
BUG=chromium:1167265
TEST=Verify unit tests pass
Change-Id: I8fc694650fe0cb883628fb566d0536d1d550df43
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2689623
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Also update the golden tests appropriately.
BUG=chromium:924760
TEST=Already tested/committed in llvm/files/compiler-wrapper.
Change-Id: I0cac1f60874d7c473a3a79e37c0eddbe3e5dfed5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2683254
Reviewed-by: George Burgess <gbiv@chromium.org>
Commit-Queue: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
|
|
Update kernel afdo profiles to R90-13754.0-1612175977.
BUG=None
TEST=None
Change-Id: I3d97bfc06fdd4db9c6606a3e99886b145a6d1eda
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2679843
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Jian Cai <jiancai@google.com>
Tested-by: Jian Cai <jiancai@google.com>
|
|
building this on the CQ is failing because this loop uses the c99
feature declaring something in the first clause of a for loop.
BUG=None
TEST=CQ
Change-Id: I92729d30bd392c46e2b1183b92874212bdc59d6f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2676930
Tested-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
In order to build this outside of the chroot, we apparently now need a
go.mod file. This is autogenerated by the `go` tool.
BUG=None
TEST=Built inside and outside of the chroot
Change-Id: I185cb9b626c8b3655ce5cc72524ef2511c7f575e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2676607
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Sometimes, I'm guilty of using ./install_compiler_wrapper.sh on broken
sources. Restoring the world to a working state requires
knowing/remembering a command. Seems better to provide that knowledge in
executable form.
BUG=None
TEST=Ran the script
Change-Id: I772b19a0b8bebdf6af78b822b710ce6e880c7388
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2676601
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Clang handles "-i686" tuple fine linking i386 libraries if needed.
"-mnomovbe" is also understood by current clang.
BUG=None
TEST=go test
Change-Id: I0d5058ebe3cadca297af1c01030480cc25194955
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2665028
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Very rarely on old GCCs, we'll complaints about GCC itself being handed
ERESTARTSYS. Retry the compilation in those cases. Similarly, we'll
sometimes see `run()` give us random errors with things like `waitid()`,
so handle those.
BUG=chromium:1166017
TEST=CQ
Change-Id: If9b6fdc523f60719608739da0eefa94c82164ae7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2648090
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Enabling this functionality with GCC is pointless -- we don't roll it,
so we don't want to catch new -Werrors with it. We're having some
GCC-specific functionality coming in soon; disabling this simplifies
things some.
BUG=chromium:1166017
TEST=CQ
Change-Id: I3f73a8124ea85a2f14fd5c909a9bbed2b46f28f4
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2648431
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Very rarely on old GCCs, we'll see an ERESTARTSYS that fails builders.
Kernel documentation indicates that users should never see ERESTARTSYS,
but it's unclear how to fix this, so try to work around it for now.
BUG=chromium:1166017
TEST=CQ
Change-Id: I5a1ca6ce722f929523feb45aeafffcc30dfda05a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2647869
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
env.exec, outside of tests and "print command" configurations, never
returns if it executed successfully (since it turns into an actual call
to execve()). Since we want to write rusage logs after the compiler is
finished, we need to not exec in that case.
BUG=None
TEST=CQ
Change-Id: If19ffb5c590868bc03431301514de875d60f7af5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2648089
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
GCC host wrapper was not calling the code to add common
flags for no clear reason. Add it so that "-fcommon" is now
passed to GCC host compiler invocations.
BUG=chromium:1171833
TEST=go test
Change-Id: Id354b65ff0a08d0b25f895bb95cd4e99412b856b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2657585
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Lexan saw a crash from clang 9.0.0 uploaded to their bucket. There's
~nothing we care to do about these crashes, and 4c even fails to parse
the header of them. Seems best to warn & ignore these cases
automatically (if an actual _user_ wants this functionality, 4c can
easily be fixed, but I doubt anyone cares)
No test is provided since submit_test_case has quite a few side-effects.
Doesn't seem worth splitting it or mocking a lot to add an automated
test for this peephole.
BUG=None
TEST=manually ran on chrotomation
Change-Id: Iae131913e7b9494015262e41af8ad4193c9d62f6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2657875
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
pass "-fcommon" to GCC to match clang. This is needed
since GCC 10 defaults to "-fno-common".
BUG=chromium:1171833
TEST=go test
Change-Id: Iceac503c8cc14afc1837c6952100189f45d2b567
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2657043
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Refactors rusage_flag and its usage in compiler_wrapper to support
the use of TOOLCHAIN_RUSAGE_OUTPUT with either FORCE_DISABLE_WERROR.
BUG=chromium:1167958
TEST=Modified and ran unit tests
Change-Id: I9a643ceabbbeca3bf3a94270d4f4b4d814812f93
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2635451
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
1. Use 'origin/main' when checking out llvm repo first time.
2. Use 'cros/main' when checking out chromiumos-overlay.
BUG=None
TEST=local bisection run
Change-Id: Ie4b2d6fcd85c730ddeec9053bb7cfc13235cca34
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2643561
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Adds current working directory as a new field in rusage logs.
This is useful for determining what package was being compiled.
BUG=chromium:1156314
TEST=Modified and ran unit tests
Change-Id: I58d7040c90fbd7d9b81532d0114862cbf4b23f88
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2636450
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Update kernel afdo profiles to R89-13710.0.
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I4b2d71695b1d8a8c3d7154f93a4e4298fffd145d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2627579
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Auto-Submit: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Tiancong Wang <tcwang@google.com>
Commit-Queue: Tiancong Wang <tcwang@google.com>
|
|
Increment a file index when copying multiple files.
This fixes the issue when multiple perf.data files are copied from
the system_health.memory_desktop benchmark with profiling.
BUG=None
TEST=crosperf with system_health.memory_desktop and profiling enabled.
Change-Id: I7d00ec23a4fa595d6cd0df8fcc8b64cf7181f5dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2622963
Tested-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Since crrev.com/c/2436432, dev-lang/rust requires
dev-lang/rust-bootstrap to build. Because of this,
rust uprevs now require that rust-bootstrap be uprevved,
too. This CL changes the rust_uprev script to do so.
BUG=chromium:1159066
TEST=python3 ./rust_tools/rust_uprev_test.py
Change-Id: I48482e780d9ebbc012142687f2edfcc8f83e9d56
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2597493
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Update chromeos-kernel-3_18 to R89-13660.0-1609151716
Update chromeos-kernel-4_4 to R89-13671.0-1609151607
Update chromeos-kernel-4_14 to R89-13662.0-1609152087
Update chromeos-kernel-4_19 to R89-13660.0-1609151538
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: Ifedee4ad33c043d87023b0a5a49050f1afdfb11a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2606609
Reviewed-by: Tiancong Wang <tcwang@google.com>
Tested-by: Denis Nikitin <denik@chromium.org>
|
|
Handling(ignoring) of "-pass-exit-codes" flag is no longer
needed in compiler wrapper.
BUG=chromium:979911
TEST=CL:2597389 passed CQ/SDK tests
Change-Id: I0114b745eed6d5dbcba10e2ecfb13d1882df32bf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2597141
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
|
|
Update chromeos-kernel-3_18 to R89-13660.0-1608546827
Update chromeos-kernel-4_4 to R89-13670.0-1608546761
Update chromeos-kernel-4_14 to R89-13662.0-1608547437
Update chromeos-kernel-4_19 to R89-13660.0-1608546981
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I14973926e5b5a6b8db1e843f6e0a0951e9ae24af
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2602676
Tested-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Tiancong Wang <tcwang@google.com>
|
|
Seeing permission errors in builder bots that create rusage_flags.
We need to set the Umask to 0 during creation of the directory.
BUG=chromium:1156314
TEST=None
Change-Id: I667d922762c2cb090adbb3abac2d4782bc48b87d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2597478
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Tiancong Wang <tcwang@google.com>
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
|
|
The ebuilds in virtual/rust can have patch (-r123) version numbers,
which currently cause rust_uprev to fail. This CL updates rust_uprev
to handle those.
BUG=chromium:1159066
TEST=python3 ./rust_tools/rust_uprev_test.py
Change-Id: I76ccaf553ea192db57e17ff3df9b02eee4105010
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2593514
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Tiancong Wang <tcwang@google.com>
|
|
Pre-submit checks complained that the mock_find_ebuild parameter to
test_success_with_template is unused. This change avoids the warning
by prefixing the paramter name with an underscore to indicate that
it is unused.
BUG=None
TEST=python3 ./rust_tools/rust_uprev_test.py
Change-Id: Ieadd3854970a1662a74159fd9e492d6ee2f5f963
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2593513
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: Tiancong Wang <tcwang@google.com>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
|
|
GETRUSAGE has been renamed TOOLCHAIN_RUSAGE_OUTPUT for clarity.
BUG=chromium:1156314
TEST=Reran unit tests for package main
Change-Id: I4588b4138f0dd4e4938762250e5caa6b4026e3f2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2594261
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
|
|
Modified rusage logs generated by GETRUSAGE flag.
Output from each entry is now in json format.
BUG=chromium:1156314
TEST=Modified and ran unit tests
Change-Id: Ia2cac4af201854de6d4a2c57e6cd8a4585f0931f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2585911
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
'target' is no longer part of the path from distfiles to chrome-src,
so this CL fixes the path inour cleanup script.
BUG=None
TEST=tested script by hand.
Change-Id: I8a69b0d153a301b61c5176e44d1ad65dc8e467cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2585652
Auto-Submit: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
The recipe builders use hyphens, while gsutil uses underscores. We
had a fix in place for this for veyron_minnie; this updates the fix for
veyron_tiger.
BUG=None
TEST=None
Change-Id: Ide32264e1cbbe29afc8ca2cd44e449b56f91a2c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2585646
Auto-Submit: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
|
|
Update chromeos-kernel-3_18 to R89-13638.0-1607337469
Update chromeos-kernel-4_4 to R89-13638.0-1607337135
Update chromeos-kernel-4_14 to R89-13638.0-1607337215
Update chromeos-kernel-4_19 to R89-13638.0-1607337429
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator.
Change-Id: I994ba9effc664cab904fccdbe0a207436ac2ea48
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2580915
Reviewed-by: Tiancong Wang <tcwang@google.com>
Commit-Queue: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
|
|
Change "tigger" to "tiger". Also update a unit test.
BUG=chromium:1156409
TEST=None.
Change-Id: I2dd394ccdfa79632e637cbefddf559e95296c8c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2580398
Reviewed-by: Tiancong Wang <tcwang@google.com>
Tested-by: Jian Cai <jiancai@google.com>
|
|
LLVM recently renamed their `master` branch to `main` upstream. This
changes our tooling to search for the latter name instead.
BUG=None
TEST=Unittests; ran `git llvm-rev --sha HEAD` in an up-to-date llvm
repo.
Change-Id: Icdc6dac48fda7697d18d0d77935e46670273bf84
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2577643
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Replace veyron_minnie with a newer model as it reaches end of life.
BUG=b:170402899
TEST=None
Change-Id: I0caf06b84b465dcc88a3fdd9dc7d19f13fcd11b9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2574734
Reviewed-by: Tiancong Wang <tcwang@google.com>
Tested-by: Jian Cai <jiancai@google.com>
|
|
The new Clang version has a number of new warnings which we disable. This CL
makes those apply to llvm (current) as well as llvm-next.
BUG=None
TEST=go test
Change-Id: I1f4929580953003d21676e7e72af4577592fb217
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2561725
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
|
|
llvm_bisection.py invokes the Gerrit CLI to abandon CLs created as part of the bisection. It passes a CL number to the command. Before this change, the CL number was passed as an int, which leads to the following error:
Traceback (most recent call last):
File "./auto_llvm_bisection.py", line 103, in main
bisection_ret = llvm_bisection.main(args_output)
File "/usr/local/google/home/inglorion/chromiumos/src/third_party/toolchain-utils/llvm_tools/llvm_bisection.py", line 364, in main
subprocess.check_output([gerrit, 'abandon', build['cl']],
File "/usr/lib/python3.8/subprocess.py", line 411, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "/usr/lib/python3.8/subprocess.py", line 489, in run
with Popen(*popenargs, **kwargs) as process:
File "/usr/lib/python3.8/subprocess.py", line 854, in __init__
self._execute_child(args, executable, preexec_fn, close_fds,
File "/usr/lib/python3.8/subprocess.py", line 1637, in _execute_child
self.pid = _posixsubprocess.fork_exec(
TypeError: expected str, bytes or os.PathLike object, not int
With this change, we convert the value to an str first, avoiding the type error.
BUG=chromium:1151325
TEST=run auto_llvm_bisection.py
Change-Id: I2b151c76757067f536712f69b14d6be549ee7c75
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2551789
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Jian Cai <jiancai@google.com>
Tested-by: Bob Haarman <inglorion@chromium.org>
|
|
Update chromeos-kernel-3_18 to R88-13577.0-1605522740
Update chromeos-kernel-4_4 to R88-13577.0-1605523015
Update chromeos-kernel-4_14 to R88-13577.0-1605522850
Update chromeos-kernel-4_19 to R88-13577.0-1605522896
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I75595037318c871b828f625b376000a6794ec65d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2542722
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Tiancong Wang <tcwang@google.com>
Tested-by: Tiancong Wang <tcwang@google.com>
|
|
Update afdo cwp kernel profiles to R88-13577 version.
BUG=None
TEST=None
Change-Id: I6acae2a8d6a47fa8f6d4a21d7b63ac75c5a0799b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2530409
Reviewed-by: Tiancong Wang <tcwang@google.com>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Ignore "core id", "apicid", "initial apicid" fields from cpuinfo in
machine checksum calculation. The values may differ on the same type of
machines.
Add more descriptive output for the BadChecksum error.
Unittest is updated accordingly.
BUG=chromium:1145386
TEST=./run_tests.sh in crosperf.
Change-Id: Ifcc91fb70f02c41d77787fbb665741bc130152c0
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2523398
Tested-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Bob Haarman <inglorion@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
|
|
Update chromeos-kernel-3_18 to R88-13560.0-1604313218
Update chromeos-kernel-4_4 to R88-13560.0-1604313145
Update chromeos-kernel-4_14 to R88-13560.0-1604313396
Update chromeos-kernel-4_19 to R88-13560.0-1604313185
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I43d2604122c5af90ed7bdb47663bcf4c1b6a59aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2523230
Tested-by: Tiancong Wang <tcwang@google.com>
Reviewed-by: Tiancong Wang <tcwang@google.com>
Commit-Queue: Tiancong Wang <tcwang@google.com>
|
|
The bug I'm chasing down right now depends on environment variables, and
could be due to a `make` invocation in one of N places. Having this
context available to me is useful.
It seriously pollutes the JSON we dump, but use of `jq` that I've put in
the doc about this makes that a non-issue.
BUG=None
TEST=Ran on stress-ng. parent process info was dumped.
Change-Id: Ib4d3b6c9887b4cf2a38472ef299c81af27e9a617
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2481829
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Update chromeos-kernel-3_18 to R88-13531.0-1603099914
Update chromeos-kernel-4_4 to R88-13531.0-1603100212
Update chromeos-kernel-4_14 to R88-13505.11-1603100505
Update chromeos-kernel-4_19 to R88-13531.0-1603100310
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: Ife0010abb801c80f1de503a88c36f342891ad0cf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2489892
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Tested-by: Bob Haarman <inglorion@chromium.org>
|