llvm-svn: 66391

llvm-svn: 66390

llvm-svn: 66389

llvm-svn: 66388

llvm-svn: 66387

llvm-svn: 66385

Patch by Jean-Daniel Dupas. Thanks!

llvm-svn: 66383

This prevents caret diagnostics from the scratch buffer from 
including other tokens in the scratch buffer that occurred beforei
them.

llvm-svn: 66375

end of line instead of just the end of buffer.  Scratch buffers contain
embedded \0's between tokens which are logic line separators.  If a 
normal text buffer contains \0's, it doesn't make a lot of sense to include
them in the caret diag output anyway.

llvm-svn: 66374

\0's to created tokens instead of making all clients do it.  No functionality
change.

llvm-svn: 66373

from being a NamedDecl to being and ObjCImplDecl.  This fixes
some valgrind issues where ObjCCategoryImpl's were being cast
to NamedDecl and then stuck on name lookup datastructures.

llvm-svn: 66372

llvm-svn: 66371

warning to only trigger when there is whitespace or something else after the + as
suggested by Eli.

llvm-svn: 66370

llvm-svn: 66356

llvm-svn: 66353

llvm-svn: 66352

llvm-svn: 66349

This is a bit cleaner, and also "fixes" bad code that compares the 
addresses of the string constants.

llvm-svn: 66346

(objc abi specific).

llvm-svn: 66345

llvm-svn: 66344

llvm-svn: 66343

correct, but an extra set of ObjC eyes would be good.

llvm-svn: 66342

llvm-svn: 66341

testing to to be sure it all works, we'll rip out all the unsupported
code.

llvm-svn: 66338

llvm-svn: 66337

llvm-svn: 66335

llvm-svn: 66334

difficult to come up with a testcase because the code generation for this
construct is broken.

llvm-svn: 66325

on all the new code by default.  There is still plenty of testing to
do and issues I'm sure need resolving.  Let me know if you find
anything.

llvm-svn: 66323

llvm-svn: 66322

performance that the type of the returned llvm::Value for an expression 
matches the converted type of the clang::Expr; mismatches will cause all
sorts of errors and silent miscompilations.

llvm-svn: 66321

llvm-svn: 66320

llvm-svn: 66319

llvm-svn: 66317

 - Shaves off a few instructions on x86_64.

One notable change: this intentionally stops setting the II->Entry
field of IdentifierInfo's retrieved by the ExternalLookup method. This
is to maintain the invariant that .getName() has a constant value for
any given IdentifierInfo. IRgen currently relies on this; which is
quite questionable but will be cleaned up in time.

Apologies for the lack of a test case; there really isn't a good way
to make one. As IRgen will eventually be fixed to not rely on this, we
can survive without one.

llvm-svn: 66316

llvm-svn: 66314

- Make Selector::getAsIdentifierInfo() private.  Using IdentifierInfo* in
  Selector is an implementation detail that clients shouldn't think about.
- Modify diagnostic emission in Sema::ProcessPropertyDecl to not use
  Selector::getAsIdentifierInfo() (which could crash when IdentifierInfo* is
  null) and instead use Selector::getAsString().
- Tidy up Selector::getAsString() implementation.

llvm-svn: 66313

PathDiagnosticControlFlowPiece to distinguish (in the class hierarchy) between
events and control-flow diagnostic pieces. Clients must now use these directly
when constructing PathDiagnosticPieces.

llvm-svn: 66310

llvm-svn: 66309

struct to an extension warning to match the behavior of GNU C, which
addresses the Sema part of PR3671.

llvm-svn: 66308