Skip to content

Commit

Permalink
[snapshot] Do not verify the checksum
Browse files Browse the repository at this point in the history
As we don't need it, and the added overhead becomes significant on large
snapshots.
  • Loading branch information
oleavr committed Oct 19, 2022
1 parent a823d7b commit 30fddd7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/snapshot/mksnapshot.cc
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ class SnapshotFileWriter {
fprintf(
fp,
"bool Snapshot::ShouldVerifyChecksum(const v8::StartupData* data) {\n");
fprintf(fp, " return true;\n");
fprintf(fp, " return false;\n");
fprintf(fp, "}\n");
fprintf(fp, "} // namespace internal\n");
fprintf(fp, "} // namespace v8\n");
Expand Down

0 comments on commit 30fddd7

Please sign in to comment.