Skip to content
  1. Feb 07, 2010
  2. Feb 06, 2010
  3. Feb 03, 2010
    • Douglas Gregor's avatar
      When a function or variable somehow depends on a type or declaration · 7dc5c17d
      Douglas Gregor authored
      that is in an anonymous namespace, give that function or variable
      internal linkage.
      
      This change models an oddity of the C++ standard, where names declared
      in an anonymous namespace have external linkage but, because anonymous
      namespace are really "uniquely-named" namespaces, the names cannot be
      referenced from other translation units. That means that they have
      external linkage for semantic analysis, but the only sensible
      implementation for code generation is to give them internal
      linkage. We now model this notion via the UniqueExternalLinkage
      linkage type. There are several changes here:
      
        - Extended NamedDecl::getLinkage() to produce UniqueExternalLinkage
          when the declaration is in an anonymous namespace.
        - Added Type::getLinkage() to determine the linkage of a type, which
          is defined as the minimum linkage of the types (when we're dealing
          with a compound type that is not a struct/class/union).
        - Extended NamedDecl::getLinkage() to consider the linkage of the
          template arguments and template parameters of function template
          specializations and class template specializations.
        - Taught code generation to rely on NamedDecl::getLinkage() when
          determining the linkage of variables and functions, also
          considering the linkage of the types of those variables and
          functions (C++ only). Map UniqueExternalLinkage to internal
          linkage, taking out the explicit checks for
          isInAnonymousNamespace().
      
      This fixes much of PR5792, which, as discovered by Anders Carlsson, is
      actually the reason behind the pass-manager assertion that causes the
      majority of clang-on-clang regression test failures. With this fix,
      Clang-built-Clang+LLVM passes 88% of its regression tests (up from
      67%). The specific numbers are:
      
      LLVM:
        Expected Passes    : 4006
        Expected Failures  : 32
        Unsupported Tests  : 40
        Unexpected Failures: 736
      
      Clang:
        Expected Passes    : 1903
        Expected Failures  : 14
        Unexpected Failures: 75
      
      Overall:
        Expected Passes    : 5909
        Expected Failures  : 46
        Unsupported Tests  : 40
        Unexpected Failures: 811
      
      Still to do:
        - Improve testing
        - Check whether we should allow the presence of types with
        InternalLinkage (in addition to UniqueExternalLinkage) given
        variables/functions internal linkage in C++, as mentioned in
        PR5792. 
        - Determine how expensive the getLinkage() calls are in practice;
        consider caching the result in NamedDecl.
        - Assess the feasibility of Chris's idea in comment #1 of PR5792.
      
      llvm-svn: 95216
      7dc5c17d
  4. Feb 01, 2010
  5. Jan 27, 2010
  6. Jan 26, 2010
  7. Jan 23, 2010
  8. Jan 10, 2010
  9. Jan 09, 2010
  10. Jan 08, 2010
  11. Jan 06, 2010
  12. Jan 05, 2010
  13. Dec 29, 2009
  14. Dec 28, 2009
  15. Dec 25, 2009
  16. Dec 12, 2009
  17. Dec 11, 2009
  18. Dec 10, 2009
  19. Dec 09, 2009
  20. Dec 08, 2009
  21. Dec 06, 2009
  22. Dec 05, 2009
  23. Dec 01, 2009
  24. Nov 30, 2009
  25. Nov 26, 2009
  26. Nov 24, 2009
  27. Nov 21, 2009
Loading