[TSan] Add missing header inclusion

llvm-svn: 177013

[TSan] Use __sanitizer_pthread_attr_t in TSan

llvm-svn: 177012

Teach X86 MC instruction lowering that VMOVAPSrr and other VEX-encoded register to register moves should be switched from using the MRMSrcReg form to the MRMDestReg form if the source register is a 64-bit extended register and the destination register is not. This allows the instruction to be encoded using the 2-byte VEX form instead of the 3-byte VEX form. The GNU assembler has similar behavior.

llvm-svn: 177011

Fix PR15309

- Fix the typo on type checking

llvm-svn: 177010

test commit: remove blank line.

llvm-svn: 177009

Remove a change to the debug info in this test, that I made while testing
something else and forgot to remove.

llvm-svn: 177007

Try using %S to find the emitted .gcno file.

llvm-svn: 177006

Remove accidentally committed debug line.

llvm-svn: 177005

Update GCOVProfiling pass creation for API change in r177002. No functionality change.

llvm-svn: 177004

Flag that friend function definitions are "late parsed" so that
template instantiation will still consider them to be definitions
if we instantiate the containing class before we get around
to parsing the friend.

This seems like a legitimate use of "late template parsed" to me,
but I'd appreciate it if someone responsible for the MS feature
would look over this.

This file already appears to access AST nodes directly, which
is arguably not kosher in the parser, but the performance of this
path matters enough that perpetuating the sin is justifiable.
Probably we ought to reconsider this policy for very simple
manipulations like this.

The reason this entire thing is necessary is that
function template instantiation plays some very gross games
in order to not associate an instantiated function template
with the class it came from unless it's a definition, and
the reason *that's* necessary is that the AST currently
cannot represent the instantiation history of individual
function template declarations, but instead tracks it in
common for the entire function template.  That probably
prevents us from correctly reporting ill-formed calls to
ambiguously instantiated friend function templates.

rdar://12350696

llvm-svn: 177003

Refactor GCOV's six constructor arguments into a struct with a getter that
constructs default arguments. It can now take default arguments from
cl::opt'ions. Add a new -default-gcov-version=... option, and actually test it!

Sink the reverse-order of the version into GCOVProfiling, hiding it from our
users.

llvm-svn: 177002

[modules] Don't write the UnusedFileScopedDecls vector to the module file.

llvm-svn: 177001

[modules] Check for delegating constructor cycles when building a module and don't write them out to the module file.

llvm-svn: 177000

Avoid computing the linkage too early. Don't invalidate it.

Before this patch we would compute the linkage lazily and cache it. When the
AST was modified in ways that could change the value, we would invalidate the
cache.

That was fairly brittle, since any code could ask for the a linkage before
the correct value was available.

We should change the API to one where the linkage is computed explicitly and
trying to get it when it is not available asserts.

This patch is a first step in that direction. We still compute the linkage
lazily, but instead of invalidating a cache, we assert that the AST
modifications didn't change the result.

llvm-svn: 176999

Added future directions comment and a couple of fixme's.

llvm-svn: 176998

Fix typo in comment.

llvm-svn: 176997

[Support][Test] Missed this in the API change.

llvm-svn: 176996

[Support] Fix lifetime of file descriptors when using MemoryBuffer.

Clients of MemoryBuffer::getOpenFile expect it not to take ownership of the file
descriptor passed in. So don't.

llvm-svn: 176995

No functionality change. Rename emitGCNO() to the more sensible
emitProfileNotes(), similar to emitProfileArcs(). Also update its comment.

Also add a comment on Version[4] (there will be another comment in clang later),
and compress lines that exceeded 80 columns.

llvm-svn: 176994

Simplify file/directory name handling in DILexicalBlock

llvm-svn: 176993

<rdar://problem/13415471>

Don't get dirty page size if we are not going to send it back

llvm-svn: 176992

Really fix the MIPS test.

llvm-svn: 176991

Remove an extra operand to a DIFile metadata entry

(extra cleanup/fallout from r176983 - not sure why I didn't catch this locally)

llvm-svn: 176988

Attempt to fix test.

llvm-svn: 176987

Reset some of the target options which affect code generation.

This doesn't reset all of the target options within the TargetOptions
object. This is because some of those are ABI-specific and must be determined if
it's okay to change those on the fly.

llvm-svn: 176986

Have these attriubtes set to 'true' or 'false'.

The back-end cannot differentiate between functions that are from a .ll file and
those generated from the front-end. We cannot then take the non-precense of
these attributes as a "false" value. Have the front-end explicitly set the value
to 'true' or 'false' depending upon what is actually set.

llvm-svn: 176985

Simplify directory name handling in DILexicalBlockFile.

llvm-svn: 176984

Remove the unused 4th operand for DIFile debug info metadata

llvm-svn: 176983

Add braces to crashed test summary (needed for buildbots to detect crashes)

llvm-svn: 176980

Remove extra whitespace

llvm-svn: 176978

Make LTO codegen use a PassManager, rather than a FunctionPassManager, for the
codegen passes. This brings it in to line with clang and llc's codegen setup,
and tidies up the code.

If I understand correctly, adding ModulePasses to a FunctionPassManager is
bogus. It only seems to explode if an added ModulePass depends on a
FunctionPass though, which might be why this code has survived so long.

Fixes <rdar://problem/13386816>.

llvm-svn: 176977

[Modules] Don't eagerly load and associate all the module header files.

In a module-enabled Cocoa PCH file, we spend a lot of time stat'ing the headers
in order to associate the FileEntries with their modules and support implicit
module import.

Use a more lazy scheme by enhancing HeaderInfoTable to store extra info about
the module that a header belongs to, and associate it with its module only when
there is a request for loading the header info for a particular file.

Part of rdar://13391765

llvm-svn: 176976

[Modules] Resolve top-headers of modules lazily.

This allows resolving top-header filenames of modules to FileEntries when
we need them, not eagerly.

Note that that this breaks ABI for libclang functions
clang_Module_getTopLevelHeader / clang_Module_getNumTopLevelHeaders
but this is fine because they are experimental and not widely used yet.

llvm-svn: 176975

Prepend "FAIL:" string to tests that crash so that buildbots pick them up.

llvm-svn: 176974

Simplify CMake rules in HandleLLVMOptions module.

Summary: No functionality change.

Reviewers: Bigcheese

Reviewed By: Bigcheese

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

llvm-svn: 176973

Minor LLDB test improvements
- fix TestCPPBool (was failing with some versions of GCC due to num_breakpoint_locations)
- speed up tests by skipping git/svn revision parsing by the test harness when running in lit-compatible (parsable) mode

This should resolve the failures reported by http://lab.llvm.org:8011/builders/lldb-x86_64-linux

llvm-svn: 176972

Cause the mips16/nomips16 attribute to be passed to LLVM from Clang
in the LLVM assembly language output.

llvm-svn: 176971

[ELF][X86_64][Hexagon] order plt/got entries properly and fix test

llvm-svn: 176970

[analyzer] BugReporter - more precise tracking of C++ references

When BugReporter tracks C++ references involved in a null pointer violation, we
want to differentiate between a null reference and a reference to a null pointer. In the
first case, we want to track the region for the reference location; in the second, we want
to track the null pointer.

In addition, the core creates CXXTempObjectRegion to represent the location of the
C++ reference, so teach FindLastStoreBRVisitor about it.

This helps null pointer suppression to kick in.

(Patch by Anna and Jordan.)

llvm-svn: 176969

Add some additonal attribute helper functions. Test will be on follow
up putback to clang for mips16.

llvm-svn: 176968

R600: Remove unused Outputs variable

llvm-svn: 176967

Remove stray space.

llvm-svn: 176966

[analyzer] Handle Objc Fast enumeration for "loop is executed 0 times".

Fixes <rdar://problem/12322528>

llvm-svn: 176965

Migrate dbg-arg.ll to a C file test to be resilient to IR changes.

llvm-svn: 176964

No functionality change. Use unreachable in getCUOffset.

llvm-svn: 176963

No functionality change. Use unreachable in getCompileUnit.

llvm-svn: 176962

<rdar://problem/13404189>

Made the "--reverse" option to "source list" also be able to use the "--count". This helps us implement support for regexp source list command:

(lldb) l -10

Which gets turned into:

(lldb) source list --reverse --count 10

Also simplified the code that is used to track showing more source from the last file and line.

llvm-svn: 176961

<rdar://problem/13396207>

Added logging that will show up in the system console when we try to resume a process that is already running, or has an unexpected state.

llvm-svn: 176960

Check register classes also when changing them.

We have the same assertion in createVirtualRegister.

llvm-svn: 176959

Add a target setting (target.use-fast-stepping) to control using the "run to next branch" stepping algorithm.

llvm-svn: 176958

Fix incorrect cast identification.

Before: int a = sizeof(int *)+ b;"
After:  int a = sizeof(int *) + b;
llvm-svn: 176957

[analyzer] fixed the logic changed by r176949

llvm-svn: 176956

ARM cost model: Add test case to make sure we would notice a change in CodeGen

In r176898 I updated the cost model to reflect the fact that sext/zext/cast on
v8i32 <-> v8i8 and v16i32 <-> v16i8 are expensive.

This test case is so that we make sure to update the cost model once we fix
CodeGen.

llvm-svn: 176955

relax check to allow for attributes (fix buildbot for elf-ppc64)

llvm-svn: 176954

Fix comment indentation before labels.

Before:
switch (x) {
  // if 1, do f()
case 1:
  f();
}

After:
switch (x) {
// if 1, do f()
case 1:
  f();
}

llvm-svn: 176953

Fix formatting issue with builder-type calls.

Before:
->aaaaaa()->aaaaaaaaaaa(aaaaaaaa()->aaaaaa()->aaaaa() - aaaaaaaaa()->aaaaaa()
                                                             ->aaaaa());

After:
a->aaaaaa()->aaaaaaaaaaa(aaaaaaaa()->aaaaaa()->aaaaa() -
                         aaaaaaaaa()->aaaaaa()->aaaaa());

llvm-svn: 176952

Added AnnotatedToken::isOneOf + a few other refactorings

Summary: <subj>

Reviewers: djasper

Reviewed By: djasper

CC: cfe-commits, klimek
Differential Revision: http://llvm-reviews.chandlerc.com/D536

llvm-svn: 176951

Fix a performance regression when combining to smaller types in icmp (shl %v, C1), C2 :

Only combine when the shl is only used by the icmp

llvm-svn: 176950

Refactoring:
+ Individual Report* method for each bug type
+ Comment improved: missing non-trivial alloca() case annotated
+ 'range' parameter of ReportBadFree() capitalized
+ 'SymbolRef Sym = State->getSVal(A, C.getLocationContext()).getAsSymbol();' shorten to 'SymbolRef Sym = C.getSVal(A).getAsSymbol();'

llvm-svn: 176949

Documentation: formatting fixes

llvm-svn: 176948

[msan] Clean up extensions during msandr shutdown

Otherwise debug DynamoRIO complains at us for leaking memory.

llvm-svn: 176947

Add one more overload to make VS2008's debug mody happy.

sigh.

llvm-svn: 176946

Reverting r176944 until Author fixes test failure.

llvm-svn: 176945

ClangTool output cleanup

Information messages sent to stdout by ClangTool now only happen when the -debug
flag is set.

Error messages that used to go to stdout now go to stderr.

Author: Ariel J Bernal <ariel.j.bernal@intel.com>
llvm-svn: 176944

[sanitizer] Fix lint.

llvm-svn: 176943

Add llvm-symbolizer as test dependency.

It is required when building tests with ASan or MSan.

llvm-svn: 176942

Add llvm-symbolizer as test dependency.

It is required when building tests with ASan or MSan.

llvm-svn: 176941

[Sanitizer] Build ASan, TSan and MSan runtimes with -fno-rtti.

llvm-svn: 176940

[msan] Increase stack size as required.

Adjust stack size in pthread_attr_t when the app-requested size does not fit MSan TLS.

llvm-svn: 176939

[sanitizer] Move GetTlsSize code from TSan to sanitizer_common.

llvm-svn: 176938

Break a >80 char line.

llvm-svn: 176937

Fix formatting of new arrays of pointers.

Before:
A = new SomeType * [Length];
A = new SomeType *[Length]();

After:
A = new SomeType *[Length];
A = new SomeType *[Length]();

Small formatting cleanups with clang-format.

llvm-svn: 176936

[Sanitizer] Fixup for r176931 for Mac and Windows

llvm-svn: 176935

Set symbolizer path in the test environment.

This is needed to get symbolized stack traces when running Clang tests under (A|M)San.

llvm-svn: 176934

Set symbolizer path in the test environment.

This is needed to get symbolized stack traces when running LLVM tests under (A|M)San.

llvm-svn: 176933

[Sanitizer] Fix a typo in function name

llvm-svn: 176932

[Sanitizer] Change MemoryMappingLayout methods to also report memory protection flags (for future use in leak checker). Patch by Sergey Matveev.

llvm-svn: 176931

lit.TestFormats.GoogleTest: Honor LitConfig.noExecute, or --no-execute were ignored.

llvm-svn: 176930

Remove trailing comma in enum list.

llvm-svn: 176926

[Pass][Layout] Add extra debugging information

llvm-svn: 176925

Tighten up the rules for precise lifetime and document
the requirements on the ARC optimizer.

rdar://13407451

llvm-svn: 176924

The step by running from branch to branch pretty much works with this checkin (at least for x86_64) but is still
turned off, it needs more qualification.  If you want to play with it, change the initialization of m_use_fast_step
to true.

llvm-svn: 176923

DoesBranch needs to compute the instruction if it isn't already done.
Handle the "alternate_isa" correctly.

llvm-svn: 176922

More cleanup, remove an untrue comment.

llvm-svn: 176921

Remove an unused #include.

llvm-svn: 176920

Don't use the fact that we stopped with a "Breakpoint" stop reason to decide to step over the breakpoint.  It's
better to check directly whether there is a breakpoint site at the PC.

llvm-svn: 176919

Add a missing check for getting a NULL oso module back from GetModuleByCompUnitInfo.

llvm-svn: 176918

[mips] Define two subclasses of MipsTargetLowering. Mips16TargetLowering is for
mips16 and MipsSETargetLowering is for mips32/64.

No functionality changes.

llvm-svn: 176917

[ELF][Writer] Let std::vector handle allocating space instead of reserving tiny portions.

llvm-svn: 176916

Various fixes for armv7 floating point/vector register support.
Drop the old f registers from debugserver's register list.  Add the
NEON 128-bit q registers to debugserver, support reading and writing.
Add the new contains / invalidates mappings for the s, d, and q
registers so lldb will know what registers overlay what other registers.
Change the default format of s and d registers to be floating point
instead of hex.  Remove some UTF-8 hyphen chars in comments in the ARM
register number definition headers.
<rdar://problem/13121797>

llvm-svn: 176915

Add a missing call to GetFileRangeMap.  It's absence was causing debug info from .o files to sometimes
not get processed.

<rdar://problem/13406310>

llvm-svn: 176914

Refactor filename/directory in DICompileUnit into a DIFile

This is the next step towards making the metadata for DIScopes have a common
prefix rather than having to delegate based on their tag type.

llvm-svn: 176913

Test for LLVM r176911

llvm-svn: 176912

Don't crash if write_impl() leaves less buffer space available than expected.

This was tickled by a Clang diagnostic; Clang test case to follow.

llvm-svn: 176911

Remove unused "isMain" field from DICompileUnit

llvm-svn: 176910

[Support][Path] Don't inf loop if creating the parent directory fails.

Patch by Paul Robinson.

llvm-svn: 176908

Update debug info test cases with empty SplitDebugFilename field.

This could be 'null' or the empty string, DIDescriptor::getStringField
coalesces the two cases anyway so it's just a matter of legible/efficient
representation.

The change in behavior of the DICompileUnit::get* functions could be
subsumed by the full verification check - but ideally that should just be an
assertion if we could front-load the actual debug info metadata failure paths.

llvm-svn: 176907

Objective-C: In my last path, also check
for existence of user setter before
issuing the warning about non-synthesizable
property. // rdar://13388503

llvm-svn: 176906

Make modules.m test portable, now that autolinking is dependent on whether or not the target uses the integrated assembler.

llvm-svn: 176902

Eliminate backend dependency in CFE testcase.

llvm-svn: 176901