AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Revert "Reland "[csa] add hint to CAST error message to break in mksnapshot""

Browse Source 
This reverts commit a10a1a65a2.

Reason: Huge test time regression, see https://crbug.com/v8/9402.

Bug: v8:9402
Change-Id: Ia71309d255ed8d9c3e96a90cc5bf6cb3ad77beaf
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1698400
Auto-Submit: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Tobias Tebbi <tebbi@chromium.org>
Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#62679}
This commit is contained in:
Michael Achenbach 2019-07-12 13:47:40 +02:00 committed by Commit Bot
parent 25aab0f3f0
commit 0ce30904c4
5 changed files with 3 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/compiler/code-assembler.cc
View File

@ -1210,13 +1210,6 @@ void CodeAssembler::HandleException(Node* node) {
raw_assembler()->AddNode(raw_assembler()->common()->IfSuccess(), node);
}
std::string CodeAssembler::CSADebugHint() {
std::stringstream s;
s << "Run mksnapshot with --csa-trap-on-node=" << state()->name() << ","
<< state()->raw_assembler_->NodeCount() << " to break in CSA code.";
return s.str();
}
namespace {
template <size_t kMaxSize>
class NodeArray {

7
src/compiler/code-assembler.h
View File

@ -747,10 +747,7 @@ class V8_EXPORT_PRIVATE CodeAssembler {
code_assembler_->SmiConstant(
static_cast<int>(ObjectTypeOf<A>::value))),
std::make_pair(MachineType::AnyTagged(),
code_assembler_->StringConstant(
(location_ + std::string("\n") +
code_assembler_->CSADebugHint())
.c_str())));
code_assembler_->StringConstant(location_)));
}
#endif
return TNode<A>::UncheckedCast(node_);
@ -1490,8 +1487,6 @@ class V8_EXPORT_PRIVATE CodeAssembler {
void GotoIfException(Node* node, Label* if_exception,
Variable* exception_var = nullptr);
std::string CSADebugHint();
// Helpers which delegate to RawMachineAssembler.
Factory* factory() const;
Isolate* isolate() const;

2
src/compiler/raw-machine-assembler.cc
View File

@ -735,8 +735,6 @@ Node* RawMachineAssembler::MakeNode(const Operator* op, int input_count,
return graph()->NewNodeUnchecked(op, input_count, inputs);
}
size_t RawMachineAssembler::NodeCount() { return graph_->NodeCount(); }
RawMachineLabel::~RawMachineLabel() {
#if DEBUG
if (bound_ == used_) return;

2
src/compiler/raw-machine-assembler.h
View File

@ -1067,8 +1067,6 @@ class V8_EXPORT_PRIVATE RawMachineAssembler {
return AddNode(op, sizeof...(args) + 1, buffer);
}
size_t NodeCount();
void SetSourcePosition(const char* file, int line);
SourcePositionTable* source_positions() { return source_positions_; }

5
test/cctest/heap/test-heap.cc
View File

@ -6523,9 +6523,8 @@ UNINITIALIZED_TEST(OutOfMemoryLargeObjects) {
}
}
CHECK_LE(state.old_generation_capacity_at_oom, kOldGenerationLimit);
size_t size = std::max(state.old_generation_capacity_at_oom,
state.memory_allocator_size_at_oom);
CHECK_LE(kOldGenerationLimit, size + state.new_space_capacity_at_oom +
CHECK_LE(kOldGenerationLimit, state.old_generation_capacity_at_oom +
state.new_space_capacity_at_oom +
state.new_lo_space_size_at_oom +
FixedArray::SizeFor(kFixedArrayLength));
CHECK_LE(