diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2022-09-02 05:47:42 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-02 03:47:42 +0000 |
| commit | d0f73d383d5e1ed58c96a272036744d4bed22b8f (patch) | |
| tree | c025c78d3c5ab7daf94a4c4cb8776032d132501f /deps/npm/node_modules/diff/lib/patch/merge.js | |
| parent | 1a0a058fdafde8b0a2aa263fc03fab34be530e6b (diff) | |
| download | node-new-d0f73d383d5e1ed58c96a272036744d4bed22b8f.tar.gz | |
build: go faster, drop -fno-omit-frame-pointer
This flag was added back in 2013 to support postmortem debugging on
Linux but it has a pretty bad impact on performance (up to 10%) and
I don't think it's actually necessary to get meaningful stack traces.
Maybe with mdb but gdb and lldb seem to manage just fine.
Even if the flag is necessary for postmortem debugging, I don't think
it's appropriate to make performance for all users suffer for the
benefit of a fringe group.
PR-URL: https://github.com/nodejs/node/pull/44452
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Diffstat (limited to 'deps/npm/node_modules/diff/lib/patch/merge.js')
0 files changed, 0 insertions, 0 deletions