Skip to content
Snippets Groups Projects
  • Jordan Rose's avatar
    d73ef135
    Add pedantic warning -Wempty-translation-unit (C11 6.9p1). · d73ef135
    Jordan Rose authored
    In standard C since C89, a 'translation-unit' is syntactically defined to have
    at least one "external-declaration", which is either a decl or a function
    definition. In Clang the latter gives us a declaration as well.
    
    The tricky bit about this warning is that our predefines can contain external
    declarations (__builtin_va_list and the 128-bit integer types). Therefore our
    AST parser now makes sure we have at least one declaration that doesn't come
    from the predefines buffer.
    
    Also, remove bogus warning about empty source files. This doesn't catch source
    files that only contain comments, and never fired anyway because of our
    predefines.
    
    PR12665 and <rdar://problem/9165548>
    
    git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@158085 91177308-0d34-0410-b5e6-96231b3b80d8
    d73ef135
    History
    Add pedantic warning -Wempty-translation-unit (C11 6.9p1).
    Jordan Rose authored
    In standard C since C89, a 'translation-unit' is syntactically defined to have
    at least one "external-declaration", which is either a decl or a function
    definition. In Clang the latter gives us a declaration as well.
    
    The tricky bit about this warning is that our predefines can contain external
    declarations (__builtin_va_list and the 128-bit integer types). Therefore our
    AST parser now makes sure we have at least one declaration that doesn't come
    from the predefines buffer.
    
    Also, remove bogus warning about empty source files. This doesn't catch source
    files that only contain comments, and never fired anyway because of our
    predefines.
    
    PR12665 and <rdar://problem/9165548>
    
    git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@158085 91177308-0d34-0410-b5e6-96231b3b80d8
Code owners
Assign users and groups as approvers for specific file changes. Learn more.