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

Issue 2707363006: [DO NOT LAND] Toggle have_futex (Closed)

Created:
3 years, 10 months ago by Tom (Use chromium acct)
Modified:
3 years, 10 months ago
CC:
chromium-reviews, Dai Mikurube (NOT FULLTIME)
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[DO NOT LAND] Toggle have_futex

Patch Set 1 #

Patch Set 2 : Add header to //base/allocator/BUILD.gn #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M base/allocator/BUILD.gn View 1 1 chunk +1 line, -0 lines 0 comments Download
M third_party/tcmalloc/chromium/src/base/spinlock_linux-inl.h View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
Tom (Use chromium acct)
+dpranke and primiano Any idea why 'analyze' determined that no compile was necessary?
3 years, 10 months ago (2017-02-23 19:52:29 UTC) #2
Dirk Pranke
On 2017/02/23 19:52:29, Tom Anderson wrote: > +dpranke and primiano > Any idea why 'analyze' ...
3 years, 10 months ago (2017-02-23 20:09:48 UTC) #3
Primiano Tucci (use gerrit)
On 2017/02/23 20:09:48, Dirk Pranke wrote: > On 2017/02/23 19:52:29, Tom Anderson wrote: > > ...
3 years, 10 months ago (2017-02-23 21:11:38 UTC) #4
Dirk Pranke
3 years, 10 months ago (2017-02-23 21:40:51 UTC) #5
On 2017/02/23 21:11:38, Primiano Tucci wrote:
> On 2017/02/23 20:09:48, Dirk Pranke wrote:
> > On 2017/02/23 19:52:29, Tom Anderson wrote:
> > > +dpranke and primiano
> > > Any idea why 'analyze' determined that no compile was necessary?
> > 
> > Looks like the header is not listed in //base/allocator/BUILD.gn
> 
> Wait, isn't analyze.py using the dependency graph generated by the compiler?
> depending on GN files seems really fragile, especially for headers which are
> added only in one .gn file but they are referenced by various translation
units
> in several targets.

No, we don't directly use the graph generated by the compiler, because in order
to get that graph we'd have to actually do the compile, which would defeat much
of the point. 

`gn check` parses c++ and header files to do a rough approximation of the
results, but it is only rough (again, to be fast).

There are discussions about how to sync things up w/ the dependency graphs from
the compilers but we haven't yet figured out a great way to do that that is both
up-to-date, too expensive, and maintainable.

Powered by Google App Engine
This is Rietveld 408576698