Skip to content
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

Jump to bottom

src: lazily initialize global BuiltinLoader #46256

Closed
wants to merge 1 commit into from
Closed

src: lazily initialize global BuiltinLoader #46256

wants to merge 1 commit into from

Commits on Jan 18, 2023

  1. src: lazily initialize global BuiltinLoader 

    Using simdutf during startup can fail when compilation units are
linked in the wrong order by a static initialization order issue.

nodejs#46235 would also solve this,
but since that is a larger effort, this patch focuses on solving
the immediate problem. Alternatively, simdutf should arguably
be made usable during static initialization, but that is also not
as straightforward as this patch.

Fixes: nodejs#46235
    @addaleax
    addaleax committed Jan 18, 2023
    Configuration menu
    Copy the full SHA
    1a2990a View commit details
    Browse the repository at this point in the history