Skip to content
Snippets Groups Projects
  • Ben Langmuir's avatar
    b1f0f8a0
    Avoid a couple of assertions when preprocessing with modules · b1f0f8a0
    Ben Langmuir authored
    1. We were hitting the NextIsPrevious assertion because we were trying
    to merge decl chains that were independent of each other because we had
    no Sema object to allow them to find existing decls. This is fixed by
    delaying loading the "preloaded" decls until Sema is available.
    
    2. We were trying to get identifier info from an annotation token, which
    asserts.  The fix is to special-case the module annotations in the
    preprocessed output printer.
    
    Fixed in a single commit because when you hit 1 you almost invariably
    hit 2 as well.
    
    git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@217550 91177308-0d34-0410-b5e6-96231b3b80d8
    b1f0f8a0
    History
    Avoid a couple of assertions when preprocessing with modules
    Ben Langmuir authored
    1. We were hitting the NextIsPrevious assertion because we were trying
    to merge decl chains that were independent of each other because we had
    no Sema object to allow them to find existing decls. This is fixed by
    delaying loading the "preloaded" decls until Sema is available.
    
    2. We were trying to get identifier info from an annotation token, which
    asserts.  The fix is to special-case the module annotations in the
    preprocessed output printer.
    
    Fixed in a single commit because when you hit 1 you almost invariably
    hit 2 as well.
    
    git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@217550 91177308-0d34-0410-b5e6-96231b3b80d8
Code owners
Assign users and groups as approvers for specific file changes. Learn more.