DescriptionRevert of Precompile more in Blink in Windows for faster compilations (patchset #9 id:160001 of https://codereview.chromium.org/1167523007/)
Reason for revert:
Strange errors in http://build.chromium.org/p/chromium.fyi/builders/Chromium%20Builder indicating that old versions of headers are used by the compiler. Possibly a stale cached pch file.
Original issue's description:
> Precompile more in Blink in Windows for faster compilations
>
> One reason Blink is slow to compile is that there is a lot of code
> included in every compilation unit since everything depends on either
> LayoutObject.h or Document.h and those in turn include huge portions
> of the rest of Blink.
>
> By precompiling LayoutObject.h and Document.h the compilation of core and
> modules in Blink can be 4 times faster (4 minutes instead of
> 19 minutes on my computer).
>
> The downside is that it will introduce Document.h and LayoutObject.h
> also in compilation units that didn't expect it, for instance
> XPathGrammer.y that suddenly will have both blink::Path and
> blink::XPath::Path in scope (and blink::Filter / blink::XPath::Filter)
>
> Note that distributed compilation system disables precompiled headers
> globally so this will *not* make trybots faster.
>
> BUG=495697
> R=thakis@chromium.org
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=198859
TBR=brucedawson@chromium.org,jam@chromium.org,scottmg@chromium.org,thakis@chromium.org,sigbjornf@opera.com,tasak@google.com,haraken@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=495697
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=199036
Patch Set 1 #
Created: 5 years, 5 months ago
(Patch set is too large to download)
Messages
Total messages: 15 (4 generated)
|