Skip to content
  1. Mar 28, 2009
  2. Mar 23, 2009
  3. Mar 22, 2009
  4. Mar 21, 2009
  5. Mar 17, 2009
  6. Mar 15, 2009
  7. Mar 14, 2009
  8. Mar 13, 2009
    • Douglas Gregor's avatar
      Refactor the way we handle operator overloading and template · 1baf54e1
      Douglas Gregor authored
      instantiation for binary operators. This change moves most of the
      operator-overloading code from the parser action ActOnBinOp to a new,
      parser-independent semantic checking routine CreateOverloadedBinOp. 
      
      Of particular importance is the fact that CreateOverloadedBinOp does
      *not* perform any name lookup based on the current parsing context (it
      doesn't take a Scope*), since it has to be usable during template
      instantiation, when there is no scope information. Rather, it takes a
      pre-computed set of functions that are visible from the context or via
      argument-dependent lookup, and adds to that set any member operators
      and built-in operator candidates. The set of functions is computed in
      the parser action ActOnBinOp based on the current context (both
      operator name lookup and argument-dependent lookup). Within a
      template, the set computed by ActOnBinOp is saved within the
      type-dependent AST node and is augmented with the results of
      argument-dependent name lookup at instantiation time (see
      TemplateExprInstantiator::VisitCXXOperatorCallExpr).
      
      Sadly, we can't fully test this yet. I'll follow up with template
      instantiation for sizeof so that the real fun can begin.
      
      llvm-svn: 66923
      1baf54e1
    • Chris Lattner's avatar
      add a helper function to strip noop casts. · ef26c77c
      Chris Lattner authored
      llvm-svn: 66909
      ef26c77c
  9. Mar 12, 2009
  10. Mar 11, 2009
  11. Mar 04, 2009
  12. Feb 27, 2009
  13. Feb 26, 2009
    • Eli Friedman's avatar
      Rewrite of isIntegerConstantExpr to be centered around Evaluate. This · 98c56a4f
      Eli Friedman authored
      is a rather big change, but I think this is the direction we want to go; 
      the code is significantly shorter now, and it doesn't duplicate Evaluate 
      code.  There shouldn't be any visible changes as far as I know.
      
      There has been some movement towards putting ICE handling into 
      Evaluate (for example, VerifyIntegerConstantExpression uses Evaluate 
      instead of isICE).  This patch is sort of the opposite of the approach, 
      making ICE handling work without Evaluate being aware of it. I think 
      this approach is better because it separates the code that does the 
      constant evaluation from code that's calculating a rather 
      arbitrary predicate.
      
      The one thing I don't really like about this patch is that 
      the handling of commas in C99 complicates it signficantly. (Seriously, 
      what was the standards committee thinking when they wrote that 
      part?) I think I've come up with a decent approach, but it doesn't feel
      ideal.  I might add some way to check for evaluated commas from Evaluate 
      in a subsequent patch; that said, it might not be worth bothering.
      
      llvm-svn: 65524
      98c56a4f
  14. Feb 24, 2009
  15. Feb 23, 2009
  16. Feb 22, 2009
  17. Feb 20, 2009
  18. Feb 18, 2009
  19. Feb 16, 2009
  20. Feb 14, 2009
    • Douglas Gregor's avatar
      Add hook to add attributes to function declarations that we know · e711f705
      Douglas Gregor authored
      about, whether they are builtins or not. Use this to add the
      appropriate "format" attribute to NSLog, NSLogv, asprintf, and
      vasprintf, and to translate builtin attributes (from Builtins.def)
      into actual attributes on the function declaration.
      
      Use the "printf" format attribute on function declarations to
      determine whether we should do format string checking, rather than
      looking at an ad hoc list of builtins and "known" function names.
      
      Be a bit more careful about when we consider a function a "builtin" in
      C++.
      
      llvm-svn: 64561
      e711f705
    • Chris Lattner's avatar
      Several related changes: · 237f2757
      Chris Lattner authored
      1) implement parser and sema support for reading and verifying attribute(warnunusedresult).
      2) rename hasLocalSideEffect to isUnusedResultAWarning, inverting the sense
         of its result.
      3) extend isUnusedResultAWarning to directly return the loc and range 
         info that should be reported to the user.  Make it substantially more
         precise in some cases than what was previously reported.
      4) teach isUnusedResultAWarning about CallExpr to decls that are 
         pure/const/warnunusedresult, fixing a fixme.
      5) change warn_attribute_wrong_decl_type to not pass in english strings, instead,
         pass in integers and use %select.
      
      llvm-svn: 64543
      237f2757
    • Douglas Gregor's avatar
      Implicitly declare certain C library functions (malloc, strcpy, memmove, · b9063fc1
      Douglas Gregor authored
      etc.) when we perform name lookup on them. This ensures that we
      produce the correct signature for these functions, which has two
      practical impacts:
      
        1) When we're supporting the "implicit function declaration" feature
        of C99, these functions will be implicitly declared with the right
        signature rather than as a function returning "int" with no
        prototype. See PR3541 for the reason why this is important (hint:
        GCC always predeclares these functions).
       
        2) If users attempt to redeclare one of these library functions with
        an incompatible signature, we produce a hard error.
      
      This patch does a little bit of work to give reasonable error
      messages. For example, when we hit case #1 we complain that we're
      implicitly declaring this function with a specific signature, and then
      we give a note that asks the user to include the appropriate header
      (e.g., "please include <stdlib.h> or explicitly declare 'malloc'"). In
      case #2, we show the type of the implicit builtin that was incorrectly
      declared, so the user can see the problem. We could do better here:
      for example, when displaying this latter error message we say
      something like:
      
        'strcpy' was implicitly declared here with type 'char *(char *, char
        const *)'
      
      but we should really print out a fake code line showing the
      declaration, like this:
      
        'strcpy' was implicitly declared here as:
      
          char *strcpy(char *, char const *)
      
      This would also be good for printing built-in candidates with C++
      operator overloading.
      
      The set of C library functions supported by this patch includes all
      functions from the C99 specification's <stdlib.h> and <string.h> that
      (a) are predefined by GCC and (b) have signatures that could cause
      codegen issues if they are treated as functions with no prototype
      returning and int. Future work could extend this set of functions to
      other C library functions that we know about.
      
      llvm-svn: 64504
      b9063fc1
  21. Feb 12, 2009
  22. Feb 09, 2009
Loading