Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(176)

Unified Diff: src/deoptimizer.h

Issue 1686183003: Use SharedFunctionInfo rather than the JSFunction in the deoptimizer (first step). (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/arm64/deoptimizer-arm64.cc ('k') | src/deoptimizer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/deoptimizer.h
diff --git a/src/deoptimizer.h b/src/deoptimizer.h
index 23813f1a3b83e06671ce5aec4e79083ae66ba758..3f6fc93c6c8ec7d1e5e37c79a2119fee9680c6aa 100644
--- a/src/deoptimizer.h
+++ b/src/deoptimizer.h
@@ -128,6 +128,11 @@ class TranslatedFrame {
Handle<SharedFunctionInfo> shared_info() const { return shared_info_; }
int height() const { return height_; }
+ SharedFunctionInfo* raw_shared_info() const {
+ CHECK_NOT_NULL(raw_shared_info_);
+ return raw_shared_info_;
+ }
+
class iterator {
public:
iterator& operator++() {
@@ -611,10 +616,10 @@ class Deoptimizer : public Malloced {
const char* debug_hint_string);
unsigned ComputeInputFrameSize() const;
- unsigned ComputeJavascriptFixedSize(JSFunction* function) const;
- unsigned ComputeInterpretedFixedSize(JSFunction* function) const;
+ static unsigned ComputeJavascriptFixedSize(SharedFunctionInfo* shared);
+ static unsigned ComputeInterpretedFixedSize(SharedFunctionInfo* shared);
- unsigned ComputeIncomingArgumentSize(JSFunction* function) const;
+ static unsigned ComputeIncomingArgumentSize(SharedFunctionInfo* shared);
static unsigned ComputeOutgoingArgumentSize(Code* code, unsigned bailout_id);
Object* ComputeLiteral(int index) const;
@@ -656,7 +661,7 @@ class Deoptimizer : public Malloced {
// Determines whether the input frame contains alignment padding by looking
// at the dynamic alignment state slot inside the frame.
- bool HasAlignmentPadding(JSFunction* function);
+ bool HasAlignmentPadding(SharedFunctionInfo* shared);
Isolate* isolate_;
JSFunction* function_;
« no previous file with comments | « src/arm64/deoptimizer-arm64.cc ('k') | src/deoptimizer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698