Node.js would not compile with --debug --v8-non-optimized-debug

Version: master
Platform: Ubuntu
Subsystem: X64 or PPC64LE

What steps will reproduce the bug?

./configure --debug --v8-non-optimized-debug
make -j32

How often does it reproduce? Is there a required condition?

Consistently

What is the expected behavior?

compilation successful

What do you see instead?

Linker fail because a ton of undefined ref
eg.

/workdir/node/out/Debug/obj/gen/torque-generated/src/objects/fixed-array-tq-inl.inc:287: undefined reference to `v8::internal::IsFixedArrayBase_NonInline(v8::internal::HeapObject)'
/usr/bin/ld: /workdir/node/out/Debug/obj.target/mksnapshot/deps/v8/src/snapshot/embedded/embedded-file-writer.o: in function `v8::internal::TorqueGeneratedByteArray<v8::internal::ByteArray, v8::internal::FixedArrayBase>::TorqueGeneratedByteArray(unsigned long)':
/workdir/node/out/Debug/obj/gen/torque-generated/src/objects/fixed-array-tq-inl.inc:349: undefined reference to `v8::internal::IsByteArray_NonInline(v8::internal::HeapObject)'
/usr/bin/ld: /workdir/node/out/Debug/obj.target/v8_base_without_compiler/deps/v8/src/api/api.o: in function `v8::internal::TorqueGeneratedPrimitiveHeapObject<v8::internal::PrimitiveHeapObject, v8::internal::HeapObject>::TorqueGeneratedPrimitiveHeapObject(unsigned long)':
/workdir/node/out/Debug/obj/gen/torque-generated/src/objects/primitive-heap-object-tq-inl.inc:4: undefined reference to `v8::internal::IsPrimitiveHeapObject_NonInline(v8::internal::HeapObject)'
/usr/bin/ld: /workdir/node/out/Debug/obj.target/v8_base_without_compiler/deps/v8/src/api/api.o: in function `v8::internal::TorqueGeneratedHeapNumber<v8::internal::HeapNumber, v8::internal::PrimitiveHeapObject>::TorqueGeneratedHeapNumber(unsigned long)':
/workdir/node/out/Debug/obj/gen/torque-generated/src/objects/heap-number-tq-inl.inc:15: undefined reference to `v8::internal::IsHeapNumber_NonInline(v8::internal::HeapObject)'
/usr/bin/ld: /workdir/node/out/Debug/obj.target/v8_base_without_compiler/deps/v8/src/api/api.o: in function `v8::internal::TorqueGeneratedJSProxy<v8::internal::JSProxy, v8::internal::JSReceiver>::TorqueGeneratedJSProxy(unsigned long)':

Additional information

Some of the torque generated .cc are not being compiled to .o eg. fixed-array-tq.o. Therefore, at the linking step, definition are missing. I suspect either gpy is not working or the GN scraper is not working as expected. I also check the Release build, fixed-array-tq.o is also missing but for some reason it compiles OK. All the torque generated .cc file are compiled to .o by GN+ninja on standalone v8 compilation.

diff --git a/tools/v8_gypfiles/v8.gyp b/tools/v8_gypfiles/v8.gyp
index 377e12d1fd..f34e4480ee 100644
--- a/tools/v8_gypfiles/v8.gyp
+++ b/tools/v8_gypfiles/v8.gyp
@@ -166,6 +166,7 @@
           '<(SHARED_INTERMEDIATE_DIR)/torque-generated/class-verifiers.h',
           '<(SHARED_INTERMEDIATE_DIR)/torque-generated/factory.cc',
           '<(SHARED_INTERMEDIATE_DIR)/torque-generated/objects-printer.cc',
+          '<@(torque_outputs_cc)'
         ],
         'include_dirs': [
           '<(SHARED_INTERMEDIATE_DIR)',

This is a potential solution. Any advice?

It does looks like the torque_generated_definitions target in the gyp file is missing the equivalent of

node/deps/v8/BUILD.gn

Lines 1618 to 1625 in e46c680

    
           foreach(file, torque_files) { 
        
             filetq = string_replace(file, ".tq", "-tq") 
        
             sources += [ 
        
               "$target_gen_dir/torque-generated/$filetq-inl.inc", 
        
               "$target_gen_dir/torque-generated/$filetq.cc", 
        
               "$target_gen_dir/torque-generated/$filetq.inc", 
        
             ] 
        
           }

@richardlau

@richardlau my patch actually works (compilation successful) but I am not sure if that's the right thing to do.

@john-yan

@john-yan Yeah, sorry, I tried a fresh checkout and both patches work. I think adding the files is the right thing to do here, I'll open a PR.

PR: #38576

build: add missing torque output sources

@richardlau

Thanks @richardlau for helping out!

build: add missing torque output sources

Navigation Menu

Repository navigation

Node.js would not compile with --debug --v8-non-optimized-debug #38571

What steps will reproduce the bug?

How often does it reproduce? Is there a required condition?

What is the expected behavior?

What do you see instead?

Additional information

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	foreach(file, torque_files) {
	filetq = string_replace(file, ".tq", "-tq")
	sources += [
	"$target_gen_dir/torque-generated/$filetq-inl.inc",
	"$target_gen_dir/torque-generated/$filetq.cc",
	"$target_gen_dir/torque-generated/$filetq.inc",
	]
	}

Node.js would not compile with --debug --v8-non-optimized-debug #38571

Description

What steps will reproduce the bug?

How often does it reproduce? Is there a required condition?

What is the expected behavior?

What do you see instead?

Additional information

Activity

john-yan commented on May 6, 2021

richardlau commented on May 6, 2021

john-yan commented on May 6, 2021

richardlau commented on May 6, 2021

richardlau commented on May 6, 2021

john-yan commented on May 7, 2021

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions