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

[iwyu] Fix allocation-site-scopes.h

Browse Source 
R=marja@chromium.org

Bug: v8:7965
Change-Id: I596712f55266c245f4df9e2fc7670e518f3e9d52
Reviewed-on: https://chromium-review.googlesource.com/c/1303294
Reviewed-by: Marja Hölttä <marja@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#57060}
This commit is contained in:
Clemens Hammacher 2018-10-27 16:51:40 +02:00 committed by Commit Bot
parent 104d75214e
commit 0fc26d8408
3 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/allocation-site-scopes-inl.h
View File

@ -12,6 +12,13 @@
namespace v8 {
namespace internal {
void AllocationSiteContext::InitializeTraversal(Handle<AllocationSite> site) {
top_ = site;
// {current_} is updated in place to not create unnecessary Handles, hence
// we initially need a separate handle.
current_ = Handle<AllocationSite>::New(*top_, isolate());
}
Handle<AllocationSite> AllocationSiteUsageContext::EnterNewScope() {
if (top().is_null()) {
InitializeTraversal(top_site_);

7
src/allocation-site-scopes.h
View File

@ -33,12 +33,7 @@ class AllocationSiteContext {
*(current_.location()) = site;
}
void InitializeTraversal(Handle<AllocationSite> site) {
top_ = site;
// {current_} is updated in place to not create unnecessary Handles, hence
// we initially need a separate handle.
current_ = Handle<AllocationSite>::New(*top_, isolate());
}
inline void InitializeTraversal(Handle<AllocationSite> site);
private:
Isolate* isolate_;

1
tools/generate-header-include-checks.py
View File

@ -29,7 +29,6 @@ AUTO_EXCLUDE = [
# flag-definitions.h needs a mode set for being included.
'src/flag-definitions.h',
# blacklist of headers we need to fix (https://crbug.com/v8/7965).
'src/allocation-site-scopes.h',
'src/compiler/allocation-builder.h',
'src/compiler/js-context-specialization.h',
'src/compiler/raw-machine-assembler.h',