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.

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

Enable LLD_REPORT_UNDEFINED by default #16003

Merged
merged 1 commit into from
Dec 8, 2022
Merged

Commits on Dec 8, 2022

  1. Enable LLD_REPORT_UNDEFINED by default

    This makes undefined symbol errors more precise by including the name of
    the object that references the undefined symbol.
    
    Its also paves the way (in my mind anyway) for finally fixing reverse
    dependencies in a salable way.  See #15982.  That PR uses an alternative
    script for the pre-processing of dependencies but also fundamentally
    relies on processing JS libraries both before and after linking.
    
    The cost is about 300ms per link operation due to double processing of
    the JS libraries, but results are cached so in practice this only
    happens the first time a given link command is run (see #18326).
    sbc100 committed Dec 8, 2022
    Configuration menu
    Copy the full SHA
    39214db View commit details
    Browse the repository at this point in the history