Reviewers: mvstanton,
Message:
Committed patchset #2 manually as r18727.
Description:
ResetAllAllocationSitesDependentCode is not changing dependent code, assert
removed.
BUG=
[email protected]
Committed: https://code.google.com/p/v8/source/detail?r=18727
Please review this at https://codereview.chromium.org/138163014/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+0, -2 lines):
M src/heap.cc
M src/objects.cc
Index: src/heap.cc
diff --git a/src/heap.cc b/src/heap.cc
index
8934acba5c0ce9ca74eb123e18108d4fac68ffc0..de912884a807df1afba7d27427829fce84d8d81b
100644
--- a/src/heap.cc
+++ b/src/heap.cc
@@ -1986,7 +1986,6 @@ void Heap::ProcessAllocationSites(WeakObjectRetainer*
retainer,
void Heap::ResetAllAllocationSitesDependentCode(PretenureFlag flag) {
- ASSERT(AllowCodeDependencyChange::IsAllowed());
DisallowHeapAllocation no_allocation_scope;
Object* cur = allocation_sites_list();
bool marked = false;
Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index
6007558cb33111a54d2e7bf6d8cf5f16cd81d8b8..94bc4271d873093a9385263ddaa36d051de0a075
100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -11799,7 +11799,6 @@ bool DependentCode::Contains(DependencyGroup group,
Code* code) {
bool DependentCode::MarkCodeForDeoptimization(
Isolate* isolate,
DependentCode::DependencyGroup group) {
- ASSERT(AllowCodeDependencyChange::IsAllowed());
DisallowHeapAllocation no_allocation_scope;
DependentCode::GroupStartIndexes starts(this);
int start = starts.at(group);
--
--
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/groups/opt_out.
