On 2015/08/17 17:51:27, domenic wrote:
On 2015/08/17 at 16:47:26, mstarzinger wrote:
>
https://codereview.chromium.org/1289603002/diff/170001/src/snapshot/natives.h
> File src/snapshot/natives.h (right):
>
>
https://codereview.chromium.org/1289603002/diff/170001/src/snapshot/natives.h#newcode9
> src/snapshot/natives.h:9: #include "src/heap/heap-inl.h"
> I have been working the entire last week to get rid of these includes.
There
is "tools/check-inline-includes.sh" which warns about these kind of
includes
(i.e. foo-inl.h included in bar.h) and I was down to 5 failures, now we
are
back
to 6. So this is somewhat disappointing at this point. :(
Yes, I saw your work and am sorry for making it harder :(. I tried
without the
includes first actually, and it compiled great on Windows, but not on the
build
bots. The whole situation is a mess and I am hopeful you can help
straighten
it
out!!
Fix for the heap-inl.h thing: https://codereview.chromium.org/1292533005/
https://codereview.chromium.org/1289603002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.