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 support of non-instrinsic debug-info #12768

Closed
wants to merge 3 commits into from
Closed

Commits on Feb 20, 2024

  1. Configuration menu
    Copy the full SHA
    b400d76 View commit details
    Browse the repository at this point in the history

Commits on Feb 21, 2024

  1. [DebugInfo][RemoveDIs] Set new-dbg-info flag from Modules correctly (…

    …#82373)
    
    It turns out there's a pathway for Functions to be inserted into modules
    without having the "New" debug-info flag set correctly, which this patch
    fixes. Sadly there isn't a Module::insert method to instrument out
    there, everyone touches the list directly.
    
    This fix exposes a path where such functions are produced in the
    outliner in the wrong mode; requiring a fix there to correctly drop
    RemoveDIs-mode debug-info. This is covered by
    test/DebugInfo/AArch64/ir-outliner.ll
    jmorse authored and jsji committed Feb 21, 2024
    Configuration menu
    Copy the full SHA
    0de50cb View commit details
    Browse the repository at this point in the history
  2. Module::getOrInsertFunction: set debug info format (#82505)

    Function::Function's constructor sets the debug info format based on the
    passed in parent Module, so by using this rather than modifying the
    function list directly, we pick up the debug info format automatically.
    hvdijk authored and jsji committed Feb 21, 2024
    Configuration menu
    Copy the full SHA
    bc2ef37 View commit details
    Browse the repository at this point in the history