[Mips] Remove unnecessary LLVM_ATTRIBUTE_UNUSED attribute.

llvm-svn: 203403

[Mips] Do not inherit MipsDynamicLibraryWriter and MipsExecutableWriter
from MipsELFWriter class. MipsELFWriter is just a helper.

llvm-svn: 203402

[Mips] Mark class by 'final' keyword.

llvm-svn: 203401

[Mips] Remove redundant 'mips' prefix from names of MipsTargetHandler
and MipsDynamicSymbolTable classes fields.

llvm-svn: 203400

[Mips] Remove redundant call to the unique_ptr::get() method followed
by the returned pointer dereferencing.

llvm-svn: 203399

[Mips] Remove unused class field.

llvm-svn: 203398

[C++11] Replacing some using declarations which are not supported on all compilers (such as MSVC 2012) with a less problematic typedef.

llvm-svn: 203397

[C++11] Use std::unique_ptr for ownership in a vector.

Also change to using range-based for loops. No functional change intended.

llvm-svn: 203396

[LCG] Simplify a bunch of the LCG code with range for loops and auto.
Still more work to be done here to leverage C++11, but this clears out
the glaring issues.

llvm-svn: 203395

[C++11] Add range views for various parts of a Module.

llvm-svn: 203394

Change documentation based on feedback from Chandler.

llvm-svn: 203393

[C++11] Now that we have C++11 and I've replaced the use of this
horrible smart pointer by std::unique_ptr and strict move semantics, rip
this out.

llvm-svn: 203392

[PM] Switch new pass manager from polymorphic_ptr to unique_ptr now that
it is available. Also make the move semantics sufficiently correct to
tolerate move-only passes, as the PassManagers *are* move-only passes.

llvm-svn: 203391

Fix build break, replace take() with release().

llvm-svn: 203390

[C++11] Replace OwningPtr include with <memory>.

llvm-svn: 203389

[C++11] Remove the remaining uses of OwningPtr.

Replace OwningArrayPtr with std::unique_ptr<T[]>.

llvm-svn: 203388

[C++11] Add llvm::make_unique, according to N3656.

llvm-svn: 203387

Revert r203230, "CodeGenPrep: sink extends of illegal types into use block."

It choked i686 stage2.

llvm-svn: 203386

Fix unwind_plan test conditions (could result in NULL memory access).

llvm-svn: 203385

PlatformWindows: Fix ObjectFile presence test conditions (could result in NULL memory access)

llvm-svn: 203384

Implement ObjectFilePECOFF::GetModuleSpecifications().

llvm-svn: 203383

[C++11] Replace OwningPtr with std::unique_ptr.

This removes all references to OwningPtr, which should be fairly
undisruptive to out-of-tree projects since they are unlikely to use
clang-tools-extra as a library instead of a set of tools.

llvm-svn: 203382

[C++11] Update to reflect the Value::use_iterator and
Value::user_iterator changes in LLVM r203364. Converts several of these
loops to nice range based loops in the process.

Built and tested cleanly for me, yay for being able to fully build and
test Polly changes!

llvm-svn: 203381

[-Wunreachable-code] Handle Objective-C bool literals in 'isConfigurationValue'.

This includes special casing 'YES' and 'NO', which are constants
defined as macros.

llvm-svn: 203380

De-virtualize some methods since they don't override anything.

llvm-svn: 203379

[C++11] Add 'override' keyword to virtual methods that override their base class.

llvm-svn: 203378

Update clang to account for changes made to LLVM in r203376

llvm-svn: 203377

IR: Change inalloca's grammar a bit

The grammar for LLVM IR is not well specified in any document but seems
to obey the following rules:

 - Attributes which have parenthesized arguments are never preceded by
   commas.  This form of attribute is the only one which ever has
   optional arguments.  However, not all of these attributes support
   optional arguments: 'thread_local' supports an optional argument but
   'addrspace' does not.  Interestingly, 'addrspace' is documented as
   being a "qualifier".  What constitutes a qualifier?  I cannot find a
   definition.

 - Some attributes use a space between the keyword and the value.
   Examples of this form are 'align' and 'section'.  These are always
   preceded by a comma.

 - Otherwise, the attribute has no argument.  These attributes do not
   have a preceding comma.

Sometimes an attribute goes before the instruction, between the
instruction and it's type, or after it's type.  'atomicrmw' has
'volatile' between the instruction and the type while 'call' has 'tail'
preceding the instruction.

With all this in mind, it seems most consistent for 'inalloca' on an
'inalloca' instruction to occur before between the instruction and the
type.  Unlike the current formulation, there would be no preceding
comma.  The combination 'alloca inalloca' doesn't look particularly
appetizing, perhaps a better spelling of 'inalloca' is down the road.

llvm-svn: 203376

Revert "Clean up SmallString a bit"

This reverts commit r203374.

Ambiguities in assign... oh well. I'm just going to revert this and
probably not try to recommit it as it's not terribly important.

llvm-svn: 203375

Clean up SmallString a bit

Move a common utility (assign(iter, iter)) into SmallVector (some of the
others could be moved there too, but this one seemed particularly
generic) and replace repetitions overrides with using directives.

And simplify SmallVector::assign(num, element) while I'm here rather
than thrashing these files (that cause everyone to rebuild) again.

llvm-svn: 203374

Remove uses of SmallString::equals in favor of SmallVectorImpl<char>'s operator==

llvm-svn: 203373

[Sema] Fix assertion hit with #pragma weak.

rdar://16264844

llvm-svn: 203372

[C++11] Fix break due to MSVC bug.

MSVC (2012, 2013, 2013 Nov CTP) fail on the following code:

int main() {
  int arr[] = {1, 2};
  for (int i : arr)
    do {} while (0);
}

The fix is to put {} around the for loop. I've reported this to the MSVC
team.

llvm-svn: 203371

[libclang] Don't pad the main buffer for the preamble.

Padding does not seem to be useful currently, and it leads to bogus location if an error
points to the end of the file.

rdar://15836513

llvm-svn: 203370

clang-tools-extra/test/pp-trace/pp-trace-modules.cpp: Flush module cache to let robust around r203317.

llvm-svn: 203369

clang-tools-extra/test/pp-trace/pp-trace-modules.cpp: Use [[@LINE]].

llvm-svn: 203368

retain-comments-from-system-headers.c: Flush module cache, or incremental test would fail since r203317.

llvm-svn: 203367

Fix build break.

llvm-svn: 203366

[C++11] Update Clang for the change to LLVM's Use-Def chain iterators in
r203364: what was use_iterator is now user_iterator, and there is
a use_iterator for directly iterating over the uses.

This also switches to use the range-based APIs where appropriate.

llvm-svn: 203365

[C++11] Add range based accessors for the Use-Def chain of a Value.

This requires a number of steps.
1) Move value_use_iterator into the Value class as an implementation
   detail
2) Change it to actually be a *Use* iterator rather than a *User*
   iterator.
3) Add an adaptor which is a User iterator that always looks through the
   Use to the User.
4) Wrap these in Value::use_iterator and Value::user_iterator typedefs.
5) Add the range adaptors as Value::uses() and Value::users().
6) Update *all* of the callers to correctly distinguish between whether
   they wanted a use_iterator (and to explicitly dig out the User when
   needed), or a user_iterator which makes the Use itself totally
   opaque.

Because #6 requires churning essentially everything that walked the
Use-Def chains, I went ahead and added all of the range adaptors and
switched them to range-based loops where appropriate. Also because the
renaming requires at least churning every line of code, it didn't make
any sense to split these up into multiple commits -- all of which would
touch all of the same lies of code.

The result is still not quite optimal. The Value::use_iterator is a nice
regular iterator, but Value::user_iterator is an iterator over User*s
rather than over the User objects themselves. As a consequence, it fits
a bit awkwardly into the range-based world and it has the weird
extra-dereferencing 'operator->' that so many of our iterators have.
I think this could be fixed by providing something which transforms
a range of T&s into a range of T*s, but that *can* be separated into
another patch, and it isn't yet 100% clear whether this is the right
move.

However, this change gets us most of the benefit and cleans up
a substantial amount of code around Use and User. =]

llvm-svn: 203364

[-Wunreachable-code] Tweak heuristic for configuration values to include arithmetic operations involving sizeof(), but not raw integers.

This case was motivated by a false positive with the
llvm::AlignOf<> specialization in LLVM.

llvm-svn: 203363

[C++11] Replacing Decl iterators attr_begin() and attr_end() with iterator_range attrs(). Updating all of the usages of the iterators with range-based for loops.

This is a reapplication of r203236 with modifications to the definition of attrs() and following the new style guidelines on auto usage.

llvm-svn: 203362

Update comment from r203315 based on review

llvm-svn: 203361

[Preprocessor] Only check for -Wunused-macros if the translation unit kind is TU_Complete.

llvm-svn: 203360

POSIX: fix possible API misuse

memcpy cannot be passed NULL.  Ensuring that the destination pointer is non-NULL
requires checking success.  Rather than performing the success check at that
point, increasing indentation an additional level, fold it into the previous
statement.

llvm-svn: 203359

POSIX: fix possible invalid API usage

strcmp cannot be passed a NULL.  Add a short-circuiting check to avoid the
possible API misuse.

llvm-svn: 203358

POSIX: add missing curly braces

It seems that the original commit missed the curly braces for the scope, always
doing the string comparision.

llvm-svn: 203357

Adding some includes to appease build bots. Amends r203354

llvm-svn: 203356

[C++11] Replacing RecordDecl iterators field_begin() and field_end() with iterator_range fields(). Updating all of the usages of the iterators with range-based for loops.

llvm-svn: 203355

Adding range-based STL-like helper APIs. llvm::distance() is the range version of std::distance. llvm::copy is the range version of std::copy.

llvm-svn: 203354

[C++11] Replacing EnumDecl iterators enumerator_begin() and enumerator_end() with iterator_range enumerators(). Updating all of the usages of the iterators with range-based for loops.

llvm-svn: 203353

Make createObjectImage and createObjectImageFromFile static methods on the
relevant subclasses of RuntimeDyldImpl. This allows construction of
RuntimeDyldImpl instances to be deferred until after the target architecture is
known.

llvm-svn: 203352

Reapplying r203299 in a slightly different manner. Now range APIs are implemented in terms of iterator APIs.

llvm-svn: 203351

Implement ObjectFilePECOFF::SetLoadAddress().

llvm-svn: 203350

Remove %zx in printf (only GCC supports it, not MSVC).

llvm-svn: 203349

Add inttypes.h to SBQueue.cpp (MSVC compilation error with PRIx32).

llvm-svn: 203348

Change else if => if after return, after r203265

llvm-svn: 203347

Fix 80 cols.

llvm-svn: 203346

[C++11] Add 'override' keyword to virtual methods that override their base class.

llvm-svn: 203345

[C++11] Add 'override' keyword to virtual methods that override their base class.

llvm-svn: 203344

[bugpoint] Don't ignore arg in -compile-commad="tool arg"

Args is an output parameter of the function lexCommand but the reference
operator was missed.

llvm-svn: 203343

[C++11] Add 'override' keyword to virtual methods that override their base class.

llvm-svn: 203342

De-virtualize a method since it doesn't override anything (yay 'override' keyword) and its class is in an anonymous namespace.

llvm-svn: 203341

[C++11] Add 'override' keyword to virtual methods that override their base class.

llvm-svn: 203340

[C++11] Add 'override' keyword to virtual methods that override their base class.

llvm-svn: 203339

Revert "Experimentally enable --verify-debug-info on some testcases and see if"
Some of the buildbots need to be upgraded to a more recen version of
dwarfdump first.

Reverting for now.

llvm-svn: 203338

DebugInfo: further improvements to test following up on r203329

llvm-svn: 203337

[-Wunreachabe-code] Don't warn about unreachable destructors for temporaries.

This can possibly be refined later, but right now the experience
is so incomprehensible for a user to understand what is going on
this isn't a useful warning.

llvm-svn: 203336

Fix CFG bug where the 'isTemporaryDtorsBranch' bit was silently lost for terminators.

llvm-svn: 203335

[CFG] Record would-be successor for noreturn destructor.

llvm-svn: 203334

[-Wunreachable-code] Handle 'return' with no argument dominated by 'noreturn' function.

llvm-svn: 203333

Need to use rm -rf on dSYM bundle directories.

llvm-svn: 203332

Remove extra space.

llvm-svn: 203331

Add API logging to the SBQueue/SBQueueItem/SBThread calls.

llvm-svn: 203330

DebugInfo: Fix test fallout from r203323

Will fix this harder in a moment.

llvm-svn: 203329

Add one more test for r203308.

llvm-svn: 203328

Experimentally enable --verify-debug-info on some testcases and see if
the buildbots can take it.

llvm-svn: 203327

Add back the original version of the test, too. (as XFAILed on Darwin)

llvm-svn: 203326

Add an option to disable plugins in clang.

An option with the same name already exists in the makefile build.

The name CLANG_IS_PRODUCTION is historical. We should probably change it, but
should change the configure build at the same time.

llvm-svn: 203325

Work around a bug in LLDB (it finds the wrong "C", rdar://problem/14933867)
and enable this test on Darwin.

llvm-svn: 203324

DebugInfo: Use DW_FORM_data4 for DW_AT_high_pc in DW_TAG_lexical_blocks

Suggested by Adrian Prantl in code review for r203187

llvm-svn: 203323

Fix a bug that mergeCases table does not match the merge constants.

MergeCases table should not have an entry for MergeContents because atoms with
MergeContents attribute should never have name. This issue was not caught by a
test because getting a value of 6th element of an array of array actually gets
the first element's value of the next array, and that happened to be a valid
value. Added asserts to catch that error.

llvm-svn: 203322

Revert "Add an option to disable plugins in clang."

This reverts commit r203320.

There is some order dependency going on that I missed.

llvm-svn: 203321

Add an option to disable plugins in clang.

An option with the same name already exists in the makefile build.

llvm-svn: 203320

Add support for hashing location information for CU level hashes.
Add a testcase based on sret.cpp where we can now hash the entire
compile unit.

llvm-svn: 203319

Revert accidentally-committed file.

llvm-svn: 203318

Module [extern_c] attribute: inherit to submodules, don't write 'extern "C"'
blocks when building in C mode, and serialize and deserialize the attribute.

llvm-svn: 203317

[DAGCombiner] Distribute TRUNC through AND in rotation amount

This is already done for shifts.  Allow it for rotations as well. E.g.:

   (rotl:i32 x, (trunc (and y, 31))) -> (rotl:i32 x, (and (trunc y), 31))

Use the newly factored-out distributeTruncateThroughAnd.

With this patch and some X86.td tweaks we should be able to remove redundant
masking of the rotation amount like in the example above.  HW implicitly
performs this masking.

The testcase will be added as part of the X86 patch.

llvm-svn: 203316

[DAGCombiner] Recognize another rotation idiom

This is the new idiom:

  x<<(y&31) | x>>((0-y)&31)

which is recognized as:

  x ROTL (y&31)

The change refines matchRotateSub.  In
Neg & (OpSize - 1) == (OpSize - Pos) & (OpSize - 1), if Pos is
Pos' & (OpSize - 1) we can just use Pos' instead of Pos.

llvm-svn: 203315

[DAGCombiner] Slightly improve readability of matchRotateSub

Slightly change the wording in the function comment. Originally, it can be
misunderstood as we turned the input into two subsequent rotates.

Better connect the comment which talks about Mask and the code which used
LoBits.  Renamed variable to MaskLoBits.

llvm-svn: 203314

[docs] Add missing features for the x86-64 ELF backend.

llvm-svn: 203313

Fix a small inferior process memory leak in SystemRuntimeMacOSX::PopulatePendingItemsForQueue().

llvm-svn: 203312

ISel: Make VSELECT selection terminate in cases where the condition type has to
be split and the result type widened.

When the condition of a vselect has to be split it makes no sense widening the
vselect and thereby widening the condition. We end up in an endless loop of
widening (vselect result type) and splitting (condition mask type) doing this.
Instead, split both the condition and the vselect and widen the result.

I ran this over the test suite with i686 and mattr=+sse and saw no regressions.

Fixes PR18036.

llvm-svn: 203311

Re-enable ProcessElfCore for non-FreeBSD/Linux builds; with Greg's fix in r203274
this is not installing itself for Mach-O binaries.

llvm-svn: 203310

Remove unnecessary test for Darwin and update testcase to be a little less
horrible/fragile.
rdar://problem/16264854

llvm-svn: 203309

[PECOFF] Support a new type of weak symbol.

Summary:
COMDAT_SELECT_SAME_SIZE is a COMDAT type that I presume exist only in COFF.
The semantics of the type is that linker should merge such COMDAT sections if
their sizes are the same. Otherwise it's an error.

Reviewers: Bigcheese, shankarke, kledzik

CC: llvm-commits
Differential Revision: http://llvm-reviews.chandlerc.com/D2996

llvm-svn: 203308

Add a virtual destructor to quiet a warning.

llvm-svn: 203307

Range-ify some for loops.

llvm-svn: 203306

Actually add the header file.

llvm-svn: 203305

Two part patch:

First: refactor out the emission of entries into the .debug_loc section
into its own routine.

Second: add a new class ByteStreamer that can be used to either emit
using an AsmPrinter or hash using DIEHash the series of bytes that
would be emitted. Use this in all of the location emission routines
for the .debug_loc section.

No functional change intended outside of a few additional comments
in verbose assembly.

llvm-svn: 203304