Skip to content

build,src: fix Intel VTune profiling support

Rodrigo Muino Tomonari requested to merge github/fork/fanchenkong1/fix-vtune-profile-support into main

Pull Request check-list

Please make sure to review and check all of these items:

  • Does make -j8 test (UNIX) or vcbuild test (Window; found some failures on Windows which also exist on main branch including parallel/test-xxx, test-wasi...) pass with this change?
  • Is the commit message formatted according to CONTRIBUTING.md?
  • If this change fixes a bug (or a performance problem), is a regression test (or a benchmark) included?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Affected core subsystem(s)

No effected core subsystem.

Description of change

This change fix the Intel Vtune profiling support for JITted JavaScript on IA32 / X64 / X32 platform. The advantage of this profiling is that the user / developer of NodeJS application can get the detailed profiling information for every line of the JavaScript source code. This information will be very useful for the owner to optimize their applications.

This feature is a compile-time option. For windows platform, the user needs to pass the following parameter to vcbuild.bat: "enable-vtune"

For other OS, the user needs to pass the following parameter to ./configure command: "--enable-vtune-profiling"

Fixes: https://github.com/nodejs/node/issues/29060

Merge request reports