Skip to content
  1. Jun 14, 2013
    • Reid Kleckner's avatar
      [Driver] Refactor clang driver to use LLVM's Option library · 898229ab
      Reid Kleckner authored 
      The big changes are:
- Deleting Driver/(Arg|Opt)*
- Rewriting includes to llvm/Option/ and re-sorting
- 'using namespace llvm::opt' in clang::driver
- Fixing the autoconf build by adding option everywhere

As discussed in the review, this change includes using directives in
header files.  I'll make follow up changes to remove those in favor of
name specifiers.

Reviewers: espindola

Differential Revision: http://llvm-reviews.chandlerc.com/D975

llvm-svn: 183989
      898229ab
    • Alexander Kornienko's avatar
      Don't remove backslashes from block comments. · be633908
      Alexander Kornienko authored 
      Summary:
Don't remove backslashes from block comments. Previously this
/* \    \ \ \ \ \
*/
would be turned to this:
/*
*/
which spoils some kinds of ASCII-art, people use in their comments. The behavior
was related to handling escaped newlines in block comments inside preprocessor
directives. This patch makes handling it in a more civilized way.

Reviewers: klimek

Reviewed By: klimek

CC: cfe-commits

Differential Revision: http://llvm-reviews.chandlerc.com/D979

llvm-svn: 183978
      be633908
    • Chandler Carruth's avatar
      Fix the warning for divide by zero to be a bit more robust. ;] · c41c8b3a
      Chandler Carruth authored 
      Previously, it only ever fired for zeros which formed null pointers.
Now, hilariously, in C++98 this was almost anything. Including tricks
like warning on the divisor in this code:

typedef char c3[3];
size_t f(c3* ptr) {
  return (sizeof(ptr) / sizeof(*ptr)) / (size_t)(!(sizeof(ptr) % sizeof(*ptr)));
}

Why the RHS of the outer divide is a null pointer constant is a sordid
tale of sorrow. Anyways, the committee fixed this for C++11 and onward
as part of core isssue 903, and Richard recently implemented this fix
causing the warning to go away here (and elsewhere).

This patch restores the warning here and adds it for numerous other
somewhat obvious gaffes:

int g(int x) {
  return x / (int)(0.0);
}

The patch is essentially just using the full power of our constant
folding in Clang to produce the warning, but insisting that it must fold
to an *integer* which is zero so that we don't get false positives
anywhere.

llvm-svn: 183970
      c41c8b3a
    • Richard Smith's avatar
      Emit initializers for static-storage-duration temporaries as constants where · a509f2fd
      Richard Smith authored 
      possible.

llvm-svn: 183967
      a509f2fd
    • Tim Northover's avatar
      Remove implicit fallthrough from AArch64 to Mips in lib selection · fbb56b7b
      Tim Northover authored 
      Nothing useful to AArch64 will (should!) be found in any Mips-specific
directories.

Patch by Luke Zarko.

llvm-svn: 183956
      fbb56b7b
  3. Jun 13, 2013
  5. Jun 12, 2013