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

PR for llvm/llvm-project#50376 #151

Open
wants to merge 1 commit into
base: release/14.x
Choose a base branch
from
Open

PR for llvm/llvm-project#50376 #151

wants to merge 1 commit into from

Commits on Apr 2, 2022

  1. [Clang] Fix Unevaluated Lambdas

    Unlike other types, when lambdas are instanciated,
    they are recreated from scratch.
    When an unevaluated lambdas appear in the type of a function,
    parameter it is instanciated in the wrong declaration context,
    as parameters are transformed before the function.
    
    To support lambda in function parameters, we try to
    compute whether they are dependant without looking at the
    declaration context.
    
    This is a short term stopgap solution to avoid clang
    iceing. A better fix might be to inject some kind of
    transparent declaration with correctly computed dependency
    for function parameters, variable templates, etc.
    
    Fixes llvm#50376
    Fixes llvm#51414
    Fixes llvm#51416
    Fixes llvm#51641
    Fixes llvm#54296
    
    Reviewed By: aaron.ballman
    
    Differential Revision: https://reviews.llvm.org/D121532
    
    (cherry picked from commit 3784e8c)
    cor3ntin authored and llvmbot committed Apr 2, 2022
    Configuration menu
    Copy the full SHA
    f42ae93 View commit details
    Browse the repository at this point in the history