Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(344)

Issue 1302053002: Roll Clang 243039:245402 (Closed)

Created:
5 years, 4 months ago by hans
Modified:
5 years, 4 months ago
Reviewers:
Nico, Adrian Kuegel
CC:
chromium-reviews, eugenis+clang_chromium.org, glider+clang_chromium.org, dmikurube+clang_chromium.org, ukai+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -138 lines) Patch
M tools/clang/plugins/FindBadConstructsConsumer.h View 1 chunk +0 lines, -6 lines 0 comments Download
M tools/clang/plugins/FindBadConstructsConsumer.cpp View 3 chunks +0 lines, -22 lines 0 comments Download
M tools/clang/scripts/package.py View 1 chunk +1 line, -0 lines 0 comments Download
M tools/clang/scripts/update.py View 3 chunks +4 lines, -54 lines 0 comments Download
M tools/clang/scripts/update.sh View 3 chunks +4 lines, -56 lines 0 comments Download

Messages

Total messages: 10 (3 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1302053002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1302053002/1
5 years, 4 months ago (2015-08-20 20:43:05 UTC) #3
Nico
On 2015/08/20 20:43:05, commit-bot: I haz the power wrote: > CQ is trying da patch. ...
5 years, 4 months ago (2015-08-20 23:36:07 UTC) #4
hans
Committed patchset #1 (id:1) manually as 80b78dfe1af1f4bb8fbc87e074c1f66d564e9d7a (presubmit successful).
5 years, 4 months ago (2015-08-20 23:44:40 UTC) #5
Adrian Kuegel
It seems this roll has made running the translation_unit tool for Codesearch horribly slow: https://uberchromegw.corp.google.com/i/chromium.infra.cron/builders/Chromium%20Linux%20Codesearch/builds/740 ...
5 years, 4 months ago (2015-08-25 14:13:29 UTC) #7
Nico
On 2015/08/25 14:13:29, Adrian Kuegel wrote: > It seems this roll has made running the ...
5 years, 4 months ago (2015-08-25 15:40:57 UTC) #8
Adrian Kuegel
On 2015/08/25 15:40:57, Nico wrote: > On 2015/08/25 14:13:29, Adrian Kuegel wrote: > > It ...
5 years, 4 months ago (2015-08-25 15:43:16 UTC) #9
Nico
5 years, 4 months ago (2015-08-25 15:47:26 UTC) #10
Message was sent while issue was closed.
On 2015/08/25 15:43:16, Adrian Kuegel wrote:
> On 2015/08/25 15:40:57, Nico wrote:
> > On 2015/08/25 14:13:29, Adrian Kuegel wrote:
> > > It seems this roll has made running the translation_unit tool for
Codesearch
> > > horribly slow:
> > >
> >
>
https://uberchromegw.corp.google.com/i/chromium.infra.cron/builders/Chromium%...
> > > 
> > > It now takes ~8 times as long as before. Any idea what might cause this?
> > 
> > Can you repro locally? Can you profile where the time goes? What does the
> > translation_unit tool do?
> 
> Will try that. What it does is to parse all files involved in compiling a
> certain cc file, to gather the information needed to recompile it
independently.

Does it use libclang, or use clang APIs some other way? Or is it just a host
program that's built by clang? I.e. is this because clang's internal AST
handling maybe got slower, or is this because clang's codegen got worse?

(Also, probably best to file a bug for this instead of discussing this here.)

Powered by Google App Engine
This is Rietveld 408576698