https://codereview.chromium.org/787033002/diff/1/Makefile
File Makefile (right):
https://codereview.chromium.org/787033002/diff/1/Makefile#newcode42
Makefile:42: GYPFLAGS += -Dv8_use_external_startup_data=1
On 2014/12/09 13:26:50, vogelheim1 wrote:
This looks like a rather surprising difference between Makefile and
gyp or GN.
Not sure that's a good idea....
Thanks for catching this. I added this line for testing only. I changed
this now so that we can use snapshot=external as make option.
https://codereview.chromium.org/787033002/diff/1/src/mksnapshot.cc
File src/mksnapshot.cc (right):
https://codereview.chromium.org/787033002/diff/1/src/mksnapshot.cc#newcode54
src/mksnapshot.cc:54: MaybeWriteStartupBlob(blob);
On 2014/12/09 13:26:50, vogelheim1 wrote:
If we have one format for both, why would we need both the raw file
and the
startup blob? Aren't they identical now?
(If so, I think the raw_file member, command line option, etc. can all
go.)
Good point. I removed the --raw-file option altogether.
https://codereview.chromium.org/787033002/diff/1/src/mksnapshot.cc#newcode68
src/mksnapshot.cc:68: void MaybeWriteRawFile(const i::Vector<const
i::byte>& blob) const {
On 2014/12/09 13:26:50, vogelheim1 wrote:
The two MaybeWrite* methods are identical except for naming. We
probably only
need one method, even if we write both files.
Done.
https://codereview.chromium.org/787033002/
--
--
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.
