| commit | 89ac34735f4819b2327983a1d34b2ca8815f87fa | [log] [tgz] |
|---|---|---|
| author | Takuto Ikuta <tikuta@chromium.org> | Thu Oct 09 09:06:51 2025 |
| committer | Copybara-Service <copybara-worker@google.com> | Thu Oct 09 09:09:15 2025 |
| tree | 47c4a90b559b68df46cc004bc649920e1f199771 | |
| parent | 749b5bfc337e10587f6a54e4689cd589d660741c [diff] |
Reland "Ignore module compile in compiler_inputs_size.py" This reverts commit 46400e56c1b814e2b28057024b3a888063873e62. Reason for revert: Fixed analyze_includes.py to handle updated commands for module compile. ``` $ led get-builder chromium/build/linux-build-perf-no-rbe | led edit-cr-cl https://chromium-review.googlesource.com/c/chromium/src/+/7015307 | led edit-system -p 30 | led launch ``` https://ci.chromium.org/ui/p/chromium/builders/build.shadow/linux-build-perf-no-rbe/37/overview Original change's description: > Revert "Ignore module compile in compiler_inputs_size.py" > > This reverts commit 17b01536a4cf9a90c1a2c1599d40f844a0ee4afe. > > Reason for revert: https://ci.chromium.org/ui/p/chromium/builders/build/linux-build-perf-no-rbe/13311/overview > > Original change's description: > > Ignore module compile in compiler_inputs_size.py > > > > This is to avoid error like > > https://ci.chromium.org/ui/p/chromium/builders/try/compile-size/277509/overview > > when we enable clang modules for compile size builder. > > > > I assume that compile size builder focuses on include and modules > > compiles are small part of build. So let me ignore module compile for > > now. > > > > Bug: 435303792 > > Change-Id: I78d9c48649de37adcbe7d207e956546b80b7ebad > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6895917 > > Reviewed-by: Hans Wennborg <hans@chromium.org> > > Auto-Submit: Takuto Ikuta <tikuta@chromium.org> > > Commit-Queue: Takuto Ikuta <tikuta@chromium.org> > > Reviewed-by: Junji Watanabe <jwata@google.com> > > Cr-Commit-Position: refs/heads/main@{#1525584} > > Bug: 435303792 > Bug: 449813885 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Change-Id: I5c6db4269b34933855e678edfa827aee9fe12134 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7015187 > Commit-Queue: Takuto Ikuta <tikuta@chromium.org> > Auto-Submit: Takuto Ikuta <tikuta@chromium.org> > Owners-Override: Takuto Ikuta <tikuta@chromium.org> > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Reviewed-by: Junji Watanabe <jwata@google.com> > Cr-Commit-Position: refs/heads/main@{#1525977} Bug: 435303792 Bug: 449813885 Change-Id: Icca595d49f7b68192ed9507d56f10cb98b4fdace Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7015307 Reviewed-by: Hans Wennborg <hans@chromium.org> Reviewed-by: Junji Watanabe <jwata@google.com> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/main@{#1527393} NOKEYCHECK=True GitOrigin-RevId: 63411c6b5caa7ad6bd389fa87e1a9ccc9c40375a