Skip to content

Nodejs v12 support #330

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
mmarchini wants to merge 9 commits into from
Closed

Nodejs v12 support #330

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
b2e6346
src: v12.x-compatible DescriptorArray
mmarchini Oct 7, 2019
8d9d9dd
test: skip inferredName on 12
mmarchini Oct 3, 2019
b164bff
src: v12 fixed for JSArrayBuffer & FixedTypedArray
mmarchini Oct 7, 2019
8491953
src: handle String__FIELD_offset__int
mmarchini Oct 8, 2019
ece9589
src: make Symbol more resilient
mmarchini Oct 8, 2019
dec5479
src: use 24 as default for kHeaderSize
mmarchini Jan 13, 2020
2d21dc8
src: guess Symbol.name offset
mmarchini Jan 14, 2020
94f2de0
build: update Node.js versions on CI matrix
mmarchini Jan 15, 2020
dd863a3
3.0.0
mmarchini Jan 16, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
src: make Symbol more resilient 
Instead of crashing if llnode fails to parse a Symbol, just return  a
??? token.
  • Loading branch information
@mmarchini
mmarchini committed Jan 16, 2020
commit ece9589264be09a9319e099703aaedef9799dac6
2 changes: 1 addition & 1 deletion src/llv8-constants.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -550,7 +550,7 @@ void Frame::Load() {


void Symbol::Load() {
kNameOffset = LoadConstant("class_Symbol__name__Object");
kNameOffset = LoadConstant({"class_Symbol__name__Object"});
}


Expand Down
2 changes: 1 addition & 1 deletion src/llv8-constants.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -509,7 +509,7 @@ class Symbol : public Module {
public:
CONSTANTS_DEFAULT_METHODS(Symbol);

int64_t kNameOffset;
Constant<int64_t> kNameOffset;

protected:
void Load();
Expand Down
2 changes: 1 addition & 1 deletion src/llv8-inl.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ ACCESSOR(Map, MaybeConstructor, map()->kMaybeConstructorOffset, HeapObject)
SAFE_ACCESSOR(Map, InstanceDescriptors, map()->kInstanceDescriptorsOffset,
HeapObject)

ACCESSOR(Symbol, Name, symbol()->kNameOffset, HeapObject)
SAFE_ACCESSOR(Symbol, Name, symbol()->kNameOffset, HeapObject)

inline int64_t Map::BitField3(Error& err) {
return v8()->LoadUnsigned(LeaField(v8()->map()->kBitField3Offset), 4, err);
Expand Down
1 change: 1 addition & 0 deletions src/llv8.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -729,6 +729,7 @@ std::string Symbol::ToString(Error& err) {
return "Symbol()";
}
HeapObject name = Name(err);
RETURN_IF_INVALID(name, "Symbol(???)");
return "Symbol('" + String(name).ToString(err) + "')";
}

Expand Down