This is something I also would like to see solved. The long phase in the middle can be even more pronounced in Chromium jumbo builds and from a high level it looks like it ought to be fixable.

I would look for a solution that does not require adding another dependency type though. Brett Wilson has stated many times that the coarse dependency system in gn is intentional because there were so many subtle dependency bugs with gyp's more fine grained dependencies.

/Daniel


On Thu, 19 Apr 2018 11:31:27 +0200, 'Takuto Ikuta' via Chromium-dev <[email protected]> wrote:



2018-04-19 17:56 GMT+09:00 Jakob Gruber <[email protected]>:
I commented on the doc. Unfortunately, I don't think it will be possible to
turn mksnapshot into a data_dep due to embedded builtins (mksnapshot
generates embedded.cc, required to build libv8.so).

Thank you for comment! Yeah, it is unfortunate that v8 generates embedded.cc. I'll reconsider the way. Introducing new dependency type like link_deps to GN might be necessary.


But looking at the graph above, it seems like splitting up code-stub-assembler.cc may be
something to investigate?


I think so. Some files in v8 took very long compile time. I hope those files are separated to utilize parallelism.

On Thu, Apr 19, 2018 at 10:48 AM, Andrew Grieve <[email protected]> wrote:
v8 build times were recently brought up as an issue (https://bugs.chromium.org/p/v8/issues/detail?id=7629), so I think your timing here is great!

I'm not on v8, but read through your proposal and it sounds good to me!

On Thu, Apr 19, 2018 at 10:09 AM, Takuto Ikuta <[email protected]> wrote:
Hi chromium folks and v8 foks.


When I build chrome, I see serialized dependency between chrome and V8 like below build trace.

This serialized dependency makes build time slow and slowness by this serialized dependency becomes relatively large in highly parallelizable >>>>build.
I want to reduce this serialized dependency by changing BUILD.gn.

More description of my proposal is here.
https://docs.google.com/document/d/1022AD3D5NtCTP0i7m-c_7JfoHu6PkJkAyePEC2ODMD8/edit?usp=sharing
If you have something, please give me feedback.

Thanks, Takuto
----Chromium Developers mailing list: [email protected]
View archives, change email options, or unsubscribe: http://groups.google.com/a/chromium.org/group/chromium-dev ---You received this message because you are subscribed to the Google Groups "Chromium-dev" group. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/chromium-dev/CALNjmMrbWafsnCbqYm>>>>%3DK9c18%2BsYzOmCqHW--rdvihZG-VVZS9g%40mail.gmail.com.

----v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---You received this message because you are subscribed to the Google Groups "v8-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
For more options, visit https://groups.google.com/d/optout.




--Takuto Ikuta
Software Engineer in Tokyo
Chrome Infrastructure (goma team)
----Chromium Developers mailing list: [email protected]
View archives, change email options, or unsubscribe: http://groups.google.com/a/chromium.org/group/chromium-dev ---You received this message because you are subscribed to the Google Groups "Chromium-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/chromium-dev/CALNjmMoDA3hyO-0G_esiG>%3Dn4k8FJu%3Dhfd_ucBG5hRusL1OAuxA%40mail.gmail.com.



--
/* Opera Software, Linköping, Sweden: CEST (UTC+2) */

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to