platform/upstream/llvm.git
4 years ago[X86] add mayRaiseFPException flag and FPCW registers for X87 instructions
Pengfei Wang [Fri, 1 Nov 2019 06:17:07 +0000 (23:17 -0700)]
[X86] add mayRaiseFPException flag and FPCW registers for X87  instructions

Summary:
This patch adds flag "mayRaiseFPException"  , FPCW and FPSW for X87 instructions which could raise
float exception.

Reviewers: pengfei, RKSimon, andrew.w.kaylor, uweigand, kpn, spatel, cameron.mcinally, craig.topper

Reviewed By: craig.topper

Subscribers: thakis, hiraditya, llvm-commits

Patch by LiuChen.

Differential Revision: https://reviews.llvm.org/D68854

4 years ago[dsymutil] Second attempt to fix dsymutil tests on non-darwin platforms
Francis Visoiu Mistrih [Sat, 2 Nov 2019 04:07:40 +0000 (21:07 -0700)]
[dsymutil] Second attempt to fix dsymutil tests on non-darwin platforms

4 years agoDon't assume that __cxa_current_exception_type exists.
Jim Ingham [Sat, 2 Nov 2019 00:20:31 +0000 (17:20 -0700)]
Don't assume that __cxa_current_exception_type exists.

Normally you shouldn't be able to have a process with an ItaniumABI plugin
that doesn't have this symbol.  But if the loader crashes before loading
libc++abi.dylib (on MacOS), then the symbol might not be present.  So we
should check before accessing the pointer.

There isn't a good way to write a test for this, but the change is obvious.

4 years ago[dsymutil] Require darwin in fat binary test
Francis Visoiu Mistrih [Fri, 1 Nov 2019 23:15:07 +0000 (16:15 -0700)]
[dsymutil] Require darwin in fat binary test

dsymutil uses lipo(1) to build the fat binary, which it invokes as a
process. For that, we need to only run this test on darwin systems.

Should fix: http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-ubuntu/builds/181/steps/test-check-all/logs/stdio

4 years ago[dsymutil] Add support for linking remarks
Francis Visoiu Mistrih [Mon, 30 Sep 2019 16:05:12 +0000 (09:05 -0700)]
[dsymutil] Add support for linking remarks

This adds support to dsymutil for linking remark files and placing them
in the final .dSYM bundle.

The result will be placed in:

* a.out.dSYM/Contents/Resources/Remarks/a.out
or
* a.out.dSYM/Contents/Resources/Remarks/a.out-<arch> for universal binaries

When multi-threaded, this runs a third thread which loops over all the
object files and parses remarks as it finds __remarks sections.

Testing this involves running dsymutil on pre-built binaries and object
files, then running llvm-bcanalyzer on the final result to check for
remarks.

Differential Revision: https://reviews.llvm.org/D69142

4 years agoDebugInfo: Accept -gdwarf even in clang-cl
David Blaikie [Fri, 1 Nov 2019 22:36:15 +0000 (15:36 -0700)]
DebugInfo: Accept -gdwarf even in clang-cl

Fixes regression introduced by llvmorg-10-init-8908-g098d901bd1b

4 years agoDebugInfo: Let -gdwarf use the toolchain default DWARF version, instead of hardcoded...
David Blaikie [Fri, 1 Nov 2019 22:17:04 +0000 (15:17 -0700)]
DebugInfo: Let -gdwarf use the toolchain default DWARF version, instead of hardcoded/aliased to -gdwarf-4

4 years ago[NewPM] Add an SROA pass after loop unroll
Guozhi Wei [Fri, 1 Nov 2019 21:59:08 +0000 (14:59 -0700)]
[NewPM] Add an SROA pass after loop unroll

If there is a small local array accessed in a loop, SROA can't handle memory
accesses with variant offset inside a loop, after the loop is fully unrolled,
all memory accesses to the array are with fixed offset, so now they can be
processed by SROA. But there is no more SROA passes after loop unroll. This
patch add an SROA pass after loop unroll to handle this pattern.

Differential Revision: https://reviews.llvm.org/D68593

4 years agoDebugInfo: Streamline debug_ranges/rnglists/rnglists.dwo emission code
David Blaikie [Fri, 1 Nov 2019 21:50:12 +0000 (14:50 -0700)]
DebugInfo: Streamline debug_ranges/rnglists/rnglists.dwo emission code

More code reuse, better basis for modelling
debug_loc/loclists/loclists.dwo emission support.

4 years agoDebugInfo: (NFC) Refactor DWARF version calculation to make a future change (-fdebug...
David Blaikie [Fri, 1 Nov 2019 21:22:29 +0000 (14:22 -0700)]
DebugInfo: (NFC) Refactor DWARF version calculation to make a future change (-fdebug-default-version) easier

4 years ago[TargetLowering] Move the setBooleanContents check on (xor (setcc), (setcc)) == ...
Craig Topper [Fri, 1 Nov 2019 21:09:08 +0000 (14:09 -0700)]
[TargetLowering] Move the setBooleanContents check on (xor (setcc), (setcc)) == / != 1 -> (setcc) != / == (setcc) to the right place

We need to be checking the value types for the inner setccs not
the outer setcc. We need to ensure those setccs produce a 0/1
value or that the xor is on the i1 type. I think at the time
this code was originally written, getBooleanContents didn't
take any arguments so this was probably correct. But now we can
have a different boolean contents for integer and floating point.

Not sure why the other combines below the xor were also checking
the boolean contents. None of them involve any setccs other than
the outer one and they only produce a new setcc.

Differential Revision: https://reviews.llvm.org/D69480

4 years ago[MachineBasicBlock] Skip over debug instructions in computeRegisterLiveness before...
Craig Topper [Fri, 1 Nov 2019 21:06:52 +0000 (14:06 -0700)]
[MachineBasicBlock] Skip over debug instructions in computeRegisterLiveness before checking for begin

If there are debug instructions before the stopping point,
we need to skip over them before checking for begin in order
to avoid having the debug instructions effect behavior.

Fixes PR43758.

Differential Revision: https://reviews.llvm.org/D69606

4 years ago[JT][CVP] Regenerate test checks, again
Nikita Popov [Fri, 1 Nov 2019 21:27:48 +0000 (22:27 +0100)]
[JT][CVP] Regenerate test checks, again

The changes to update_test_checks format have been disabled again,
so regenerate these tests. Also regenerate select.ll.

4 years ago[amdgpu] Fix known bits compuation on `MUL_I24`/`MUL_U24`.
Michael Liao [Fri, 1 Nov 2019 19:49:41 +0000 (15:49 -0400)]
[amdgpu] Fix known bits compuation on `MUL_I24`/`MUL_U24`.

Reviewers: arsenm, rampitec

Subscribers: kzhuravl, jvesely, wdng, nhaehnle, dstuttard, tpr, t-tye, hiraditya, llvm-commits, yaxunl

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69735

4 years agoRecommit "[ThinLTO] Handle GUID collision in import global processing""
Teresa Johnson [Fri, 1 Nov 2019 17:31:02 +0000 (10:31 -0700)]
Recommit "[ThinLTO] Handle GUID collision in import global processing""

This recommits cc0b9647b76178bc3869bbfff80535ad86366472 which was
reverted in d39d1a2f87aca3cfabe58ecfa5146879baa70096.

I added a fix for an issue found when testing via distributed ThinLTO,
and added a test case for that failure.

4 years ago[WinCFG] Handle constant casts carefully in .gfids emission
Reid Kleckner [Wed, 30 Oct 2019 23:32:26 +0000 (16:32 -0700)]
[WinCFG] Handle constant casts carefully in .gfids emission

Summary:
The general Function::hasAddressTaken has two issues that make it
inappropriate for our purposes:
1. it is sensitive to dead constant users (PR43858 / crbug.com/1019970),
   leading to different codegen when debu info is enabled
2. it considers direct calls via a function cast to be address escapes

The first is fixable, but the second is not, because IPO clients rely on
this behavior. They assume this function means that all call sites are
analyzable for IPO purposes.

So, implement our own analysis, which gets closer to finding functions
that may be indirect call targets.

Reviewers: ajpaverd, efriedma, hans

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69676

4 years ago[NFC]: Removed an implicit capture argument from lambda.
Sourabh Singh Tomar [Fri, 1 Nov 2019 19:58:40 +0000 (01:28 +0530)]
[NFC]: Removed an implicit capture argument from lambda.

4 years ago[X86] Change the behavior of canWidenShuffleElements used by lowerV2X128Shuffle to...
Craig Topper [Fri, 1 Nov 2019 18:28:32 +0000 (11:28 -0700)]
[X86] Change the behavior of canWidenShuffleElements used by lowerV2X128Shuffle to match the behavior in lowerVectorShuffle with regards to zeroable elements.

Previously we marked zeroable elements in a way that prevented
the widening check from recognizing that it could widen. Now
we only mark them zeroable if V2 is an all zeros vector. This
matches what we do for widening elements in lowerVectorShuffle.

Fixes PR43866.

4 years agoWarn when an output section name is longer than 8 characters
Reid Kleckner [Tue, 29 Oct 2019 22:57:40 +0000 (15:57 -0700)]
Warn when an output section name is longer than 8 characters

Recent versions of Microsoft's dumpbin tool cannot handle such PE files.
LLVM tools and GNU tools can, and use this to encode long section names
like ".debug_info", which is commonly used for DWARF. Don't do this in
mingw mode or when -debug:dwarf is passed, since the user probably wants
long section names for DWARF sections.

PR43754

Reviewers: ruiu, mstorsjo

Differential Revision: https://reviews.llvm.org/D69594

4 years ago[CMake] Add cross Windows to ARM Linux toolchain CMake cache file.
Vladimir Vereschaka [Fri, 1 Nov 2019 18:52:27 +0000 (11:52 -0700)]
[CMake] Add cross Windows to ARM Linux toolchain CMake cache file.

This cache file can be used to build a cross Windows to ARM Linux
toolchain.

Differential Revision: https://reviews.llvm.org/D69651

4 years agoReapply [LLDB] [test] Use %clang_cl instead of build.py in a few tests
Martin Storsjö [Wed, 16 Oct 2019 12:11:50 +0000 (15:11 +0300)]
Reapply [LLDB] [test] Use %clang_cl instead of build.py in a few tests

This allows explicitly specifying the intended target architecture,
for tests that aren't supposed to be executed, and that don't
require MSVC headers or libraries to be available.

(These tests already implicitly assumed to be built for x86; one
didn't specify anything, assuming x86_64, while the other specified
--arch=32, which only picks the 32 bit variant of the default target
architecture).

Join two comment lines in disassembly.cpp, to keep row numbers
checked in the test unchanged.

This fixes running check-lldb on arm linux.

Previously when this was applied (in 95980409e6), it broke
macos buildbots, as they added "-isysroot <path>" to all %clang*
substitutions, and clang-cl didn't support that.

Reapplying it without further changes to this patch, after D69619
(9c73925226), because now, such extra parameters are added to
%clang_host*, but not to plain %clang_cl.

Differential Revision: https://reviews.llvm.org/D69031

4 years ago[debuginfo-tests] Don't look for Python 3 if we already have it
Reid Kleckner [Thu, 31 Oct 2019 21:00:22 +0000 (14:00 -0700)]
[debuginfo-tests] Don't look for Python 3 if we already have it

LLDB already requires Python 3 on Windows, so I already configure it
that way. For some reason CMake fails to find the one that Visual Studio
automatically installs at this standard location:
  C:/Program Files (x86)/Microsoft Visual Studio/Shared/Python37_64/python.exe

CMake prefers the python on path, which happens to be python 2.7.

Reviewers: aprantl, jmorse

Differential Revision: https://reviews.llvm.org/D69684

4 years ago[lit] Fix internal env calling env
Joel E. Denny [Fri, 1 Nov 2019 14:14:22 +0000 (10:14 -0400)]
[lit] Fix internal env calling env

Without this patch, when using lit's internal shell, if `env` on a lit
RUN line calls `env`, lit accidentally searches for the latter as an
external executable.  What's worse is that works fine when a developer
is testing on a platform where `env` is available and behaves as
expected, but it then breaks on other platforms.

`env` calling `env` can make sense if one such `env` is within a lit
substitution, as in D65156 and D65121.  This patch ensures that lit
executes both as internal commands.

Reviewed By: probinson, mgorny, rnk

Differential Revision: https://reviews.llvm.org/D65697

4 years ago[MIPS GlobalISel] Fix -Wunused-variable in -DLLVM_ENABLE_ASSERTIONS=off builds after...
Fangrui Song [Fri, 1 Nov 2019 18:07:21 +0000 (11:07 -0700)]
[MIPS GlobalISel] Fix -Wunused-variable in -DLLVM_ENABLE_ASSERTIONS=off builds after D69663

4 years agoIllustrate a redirector using the example of round function from math.h.
Siva Chandra Reddy [Tue, 15 Oct 2019 20:25:36 +0000 (13:25 -0700)]
Illustrate a redirector using the example of round function from math.h.

Setup demonstrated in this patch is only for ELF-ish platforms.

Also note:

1. Use of redirectors is a temporary scheme. They will be removed once
   LLVM-libc has implementations for the redirected functions.
2. Redirectors are optional. One can choose to not include them in the
   LLVM-libc build for their platform.
3. Even with redirectors used, we want to link to the system libc
   dynamically.

Reviewers: dlj, hfinkel, jakehehrlich, phosek, stanshebs, theraven, alexshap

Subscribers: mgorny, libc-commits

Tags: #libc-project

Differential Revision: https://reviews.llvm.org/D69020

4 years ago[X86][AVX] Add support for and/or scalar bool reduction with AVX512 mask registers
Simon Pilgrim [Fri, 1 Nov 2019 17:55:18 +0000 (17:55 +0000)]
[X86][AVX] Add support for and/or scalar bool reduction with AVX512 mask registers

combineBitcastvxi1 only handles bitcast->MOVMSK combines, with mask registers we use BITCAST directly.

4 years ago[dsymutil] Add DW_TAG_common_block to dieNeedsChildrenToBeMeaningful
Jonas Devlieghere [Fri, 1 Nov 2019 17:43:39 +0000 (10:43 -0700)]
[dsymutil] Add DW_TAG_common_block to dieNeedsChildrenToBeMeaningful

Ensure we walk the children of common blocks when deciding what DIEs to
keep. Otherwise we might incorrectly discard them leading to missing
variables in the linked debug info.

This also sorts the list of DW_TAGs alphabetically.

4 years agoAdd MemTagSanitizer documentation.
Evgenii Stepanov [Fri, 18 Oct 2019 19:32:21 +0000 (12:32 -0700)]
Add MemTagSanitizer documentation.

Summary: A lot of this is work in progress...

Reviewers: kcc, pcc

Subscribers: cryptoad, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69289

4 years ago[WebAssembly] Add experimental SIMD dot product instruction
Thomas Lively [Fri, 1 Nov 2019 01:28:02 +0000 (18:28 -0700)]
[WebAssembly] Add experimental SIMD dot product instruction

Summary:
This instruction is not merged to the spec proposal, but we need it to
be implemented in the toolchain to experiment with it. It is available
only on an opt-in basis through a clang builtin.

Defined in https://github.com/WebAssembly/simd/pull/127.

Depends on D69696.

Reviewers: aheejin

Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, cfe-commits, llvm-commits

Tags: #clang, #llvm

Differential Revision: https://reviews.llvm.org/D69697

4 years ago[LV] Move interleave_short_tc.ll into the X86 directory to hopefully make fix non...
Craig Topper [Fri, 1 Nov 2019 17:18:54 +0000 (10:18 -0700)]
[LV] Move interleave_short_tc.ll into the X86 directory to hopefully make fix non-X86 bots.

4 years agoReland "[WebAssembly] Expand setcc of v2i64"
Thomas Lively [Fri, 1 Nov 2019 17:21:00 +0000 (10:21 -0700)]
Reland "[WebAssembly] Expand setcc of v2i64"

This reverts commit e5cae5692b5899631b5bfe5c23234deb5efda10c, which
reverted 11850a6305c5778b180243eb06aefe86762dd4ce. The original revert
was done because of breakage that was actually in a separate commit,
2ab1b8c1ec452fb743f6cc5051e75a01039cabfe, which was also reverted and
has since been fixed and relanded.

4 years ago[Support] Use /proc/self/exe for GNU Hurd
kristina [Fri, 1 Nov 2019 17:27:24 +0000 (17:27 +0000)]
[Support] Use /proc/self/exe for GNU Hurd

Use `/proc/self/exe` to get the current executable
path on GNU Hurd.

Patch by sthibaul (Samuel Thibault)

Differential Revision: https://reviews.llvm.org/D69683

4 years ago[X86] isFNEG - use switch() instead of if-else tree. NFCI.
Simon Pilgrim [Fri, 1 Nov 2019 17:08:39 +0000 (17:08 +0000)]
[X86] isFNEG - use switch() instead of if-else tree. NFCI.

In a future patch this will avoid some checks which don't need to be done for some opcodes.

4 years agoDisable exceptions in libfuzzer's copy of libcxxabi.
Evgenii Stepanov [Fri, 1 Nov 2019 17:04:38 +0000 (10:04 -0700)]
Disable exceptions in libfuzzer's copy of libcxxabi.

External project configuration for libcxxabi now has exceptions on by
default, but this is not needed for libfuzzer.

4 years agoRevert "[LLD][ThinLTO] Handle GUID collision in import global processing"
Teresa Johnson [Fri, 1 Nov 2019 16:43:06 +0000 (09:43 -0700)]
Revert "[LLD][ThinLTO] Handle GUID collision in import global processing"

This reverts commit cc0b9647b76178bc3869bbfff80535ad86366472.

The commit is causing a failure in internal testing. Will recommit with
a fix later.

4 years ago[X86] Fix -DBUILD_SHARED_LIBS=on builds after D69568/llvmorg-10-init-8877-g3a399c09878
Fangrui Song [Fri, 1 Nov 2019 16:45:27 +0000 (09:45 -0700)]
[X86] Fix -DBUILD_SHARED_LIBS=on builds after D69568/llvmorg-10-init-8877-g3a399c09878

In -DBUILD_SHARED_LIBS=on builds, a component must specify its direct dependencies to satisfy -Wl,-z,defs (added by llvm/modules/HandleLLVMOptions.cmake).

Core is a direct dependency via transitive header inclusion:

  ld.lld: error: undefined symbol: llvm::LLVMContext::LLVMContext()
  >>> referenced by MachineSizeOptsTest.cpp
  >>>               unittests/Target/X86/CMakeFiles/X86Tests.dir/MachineSizeOptsTest.cpp.o:(testing::internal::TestFactoryImpl<(anonymous namespace)::MachineSizeOptsTest_Test_Test>::CreateTest())

MC is a direct dependency via transitive header inclusion:

  ld.lld: error: undefined symbol: llvm::MCTargetOptions::MCTargetOptions()
  >>> referenced by MachineSizeOptsTest.cpp
  >>>               unittests/Target/X86/CMakeFiles/X86Tests.dir/MachineSizeOptsTest.cpp.o:((anonymous namespace)::MachineSizeOptsTest::SetUp())

4 years ago[Utils] Hide the default behavior change of D68819 under a flag
Johannes Doerfert [Fri, 1 Nov 2019 16:17:27 +0000 (11:17 -0500)]
[Utils] Hide the default behavior change of D68819 under a flag

With D69701, the options used when running the script on a file will be
recorded and reused on a rerun. This allows us to hide new features
behind flags, starting with the "define" that was introduced in D68819.

4 years agoRemove unnecessary unit test dependencies.
Hiroshi Yamauchi [Tue, 29 Oct 2019 16:23:14 +0000 (09:23 -0700)]
Remove unnecessary unit test dependencies.

Summary: This is to address comment on D69409.

Reviewers: davidxl, thakis

Subscribers: mgorny, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69568

4 years agollvm-config: emit the C++ standard flag into CXXFLAGS
Saleem Abdulrasool [Fri, 1 Nov 2019 16:12:03 +0000 (09:12 -0700)]
llvm-config: emit the C++ standard flag into CXXFLAGS

This recovers the now "missing" flag as this is controlled by CMake
rather than injected into the user defined flags list.  This is
primarily needed by LDC and other out-of-tree users which do not
correctly setup the C++ flags.

4 years ago[InstCombine] regenerate test checks; NFC
Sanjay Patel [Fri, 1 Nov 2019 16:07:09 +0000 (12:07 -0400)]
[InstCombine] regenerate test checks; NFC

Avoid subsequent test noise from improved CHECK-LABEL matching.

4 years agoRevert "[AArch64][MachineOutliner] Return address signing for outlined functions"
Oliver Stannard [Fri, 1 Nov 2019 16:00:07 +0000 (16:00 +0000)]
Revert "[AArch64][MachineOutliner] Return address signing for outlined functions"

This is causing faults when an instruction which modifies SP is
outlined, causing the PAC and AUT instructions to not match.

This reverts commits 70caa1fc30c392974df3bccd9959765dae1779f6 and
55314d323738e4a8c1890b6a6e5064e7f4e0da1c.

4 years ago[libTooling] Add Stencil constructor.
Yitzhak Mandelbaum [Wed, 30 Oct 2019 18:57:52 +0000 (14:57 -0400)]
[libTooling] Add Stencil constructor.

Summary:
Adds a constructor that takes a vector with which to initialize the `Parts`
field and a corresponding free function that forwards to the constructor. These
definitions are needed to assist in transitioning away from `Stencil` as a class
to defining it as a type alias.

Reviewers: ilya-biryukov

Subscribers: cfe-commits, gribozavr

Tags: #clang

Differential Revision: https://reviews.llvm.org/D69632

4 years ago[libomptarget] Implement target_impl for amdgcn
JonChesterfield [Fri, 1 Nov 2019 14:58:39 +0000 (14:58 +0000)]
[libomptarget] Implement target_impl for amdgcn

Summary:
[libomptarget] Implement target_impl for amdgcn

Smallest atomic addition for a new target. Implements enough of the amdgcn
specific code that some of the source files under nvptx/src could be compiled,
without modification, to run on amdgcn.

This foreshadows a work in progress patch to move said source out of nvptx/src.
Patch based on fork at https://github.com/ROCm-Developer-Tools/llvm-project

Reviewers: ABataev, jdoerfert, grokos, ronlieb

Subscribers: jvesely, jfb, openmp-commits

Tags: #openmp

Differential Revision: https://reviews.llvm.org/D69718

4 years ago[LDV][RAGreedy] Inform LiveDebugVariables about new VRegs added by InlineSpiller
Bjorn Pettersson [Fri, 25 Oct 2019 17:03:18 +0000 (19:03 +0200)]
[LDV][RAGreedy] Inform LiveDebugVariables about new VRegs added by InlineSpiller

Summary:
Make sure RAGreedy informs LiveDebugVariables about new VRegs
that is introduced at spill by InlineSpiller.

Consider this example

 LDV: !"var"  [48r;128r):0 Loc0=%2

 48B   %2 = ...
 ...
 128B  %7 = ADD %2, ...

If %2 is spilled the InlineSpiller will insert spill/reload
instructions and introduces some new vregs. So we get

 48B   %4 = ...
 56B   spill %4
 ...
 120B  reload %5
 128B  %3 = ADD %5, ...

In the past we did not inform LDV about this, and when reintroducing
DBG_VALUE instruction LDV still got information that "var" had the
location of the spilled register %2 for the interval [48r;128r).
The result was bad, since we mapped "var" to the spill slot even
before the spill happened:

 %4 = ...
 DBG_VALUE %spill.0, !"var"
 spill %4 to %spill.0
 ...
 reload %5
 %3 = ADD %5, ...

This patch will inform LDV about the interval split introduced
due to spilling. So the location map in LDV will become

 !"var" [48r;56r):1 [56r;120r):0 [120r;128r):2 Loc0=%2 Loc1=%4 Loc2=%5

And when inserting DBG_VALUE instructions we get

 %4 = ...
 DBG_VALUE %4, !"var"
 spill %4 to %spill.0
 DBG_VALUE %spill.0, !"var"
 ...
 reload %5
 DBG_VALUE %5, !"var"
 %3 = ADD %5, ...

Fixes: https://bugs.llvm.org/show_bug.cgi?id=38899

Reviewers: jmorse, vsk, aprantl

Reviewed By: jmorse

Subscribers: dstenb, wuzish, MatzeB, qcolombet, nemanjai, hiraditya, jsji, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69584

4 years agogn build: Merge d36a0333102
LLVM GN Syncbot [Fri, 1 Nov 2019 15:17:32 +0000 (15:17 +0000)]
gn build: Merge d36a0333102

4 years ago[clang-tidy] New checker performance-trivially-destructible-check
Anton Bikineev [Fri, 25 Oct 2019 11:03:53 +0000 (13:03 +0200)]
[clang-tidy] New checker performance-trivially-destructible-check

Checks for types which can be made trivially-destructible by removing
out-of-line defaulted destructor declarations.

The check is motivated by the work on C++ garbage collector in Blink
(rendering engine for Chrome), which strives to minimize destructors and
improve runtime of sweeping phase.

In the entire chromium codebase the check hits over 2000 times.

Differential Revision: https://reviews.llvm.org/D69435

4 years ago[AArch64] Output the pseudo SPACE in asm and object files
Momchil Velikov [Fri, 1 Nov 2019 15:01:36 +0000 (15:01 +0000)]
[AArch64] Output the pseudo SPACE in asm and object files

Summary: It outputs nothing, but is useful for writing tests, checking asm output.

Reviewers: t.p.northover, ostannard, tellenbach

Reviewed By: tellenbach

Subscribers: tellenbach, kristof.beyls, hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69185

Change-Id: I6b58310e9e5632f0976d2000ce975ee28df90ebe

4 years ago[Object] Remove extra space in error message
James Henderson [Fri, 1 Nov 2019 14:07:34 +0000 (14:07 +0000)]
[Object] Remove extra space in error message

Previously this message had a double space in it.

4 years ago[Dexter] Continue sprinking no-location fixes
Jeremy Morse [Fri, 1 Nov 2019 13:56:31 +0000 (13:56 +0000)]
[Dexter] Continue sprinking no-location fixes

Example failure:

http://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/3255/testReport/junit/debuginfo-tests/dexter-tests/asan_c/

The tests themselves seem to be working, it's just unexplored paths within
dexter that are flaking out.

4 years ago[OpenCL] Support -fdeclare-opencl-builtins in C++ mode
Sven van Haastregt [Fri, 1 Nov 2019 13:56:43 +0000 (13:56 +0000)]
[OpenCL] Support -fdeclare-opencl-builtins in C++ mode

Support for C++ mode was accidentally lacking due to not checking the
OpenCLCPlusPlus LangOpts version.

Differential Revision: https://reviews.llvm.org/D69233

4 years ago[LIBOMPTARGET]Call GetLaneId function, do not use its address in debug
Alexey Bataev [Fri, 1 Nov 2019 13:42:46 +0000 (09:42 -0400)]
[LIBOMPTARGET]Call GetLaneId function, do not use its address in debug
log functions.

4 years ago[Dexter] Account for another no-lineno scenario
Jeremy Morse [Fri, 1 Nov 2019 13:41:56 +0000 (13:41 +0000)]
[Dexter] Account for another no-lineno scenario

This is another part of Dexter that had never seen a missing source
location before, now newly turning up on Darwin.

4 years ago[Dexter] Cope better with empty source locations
Jeremy Morse [Fri, 1 Nov 2019 13:20:26 +0000 (13:20 +0000)]
[Dexter] Cope better with empty source locations

When running a program, Dexter single steps if it's in one of the source
files under test, or free-runs if it isn't. Handle the circumstance where
the current source file simply isn't known.

4 years ago[SLP] avoid 'tmp' value name conflict with auto-generated CHECK script; NFC
Sanjay Patel [Fri, 1 Nov 2019 13:27:20 +0000 (09:27 -0400)]
[SLP] avoid 'tmp' value name conflict with auto-generated CHECK script; NFC

The script uses 'TMP#' as its substitute for nameless values,
so if a test already contains 'tmp#' *named* values, then
there could be trouble. We should probably just fix the
script to avoid this problem going forward, but it's easy
enough to change a test too (and explicitly naming variables
'tmp' is always a sad choice).

4 years ago[SLP] avoid 'tmp' value name conflict with auto-generated CHECK script; NFC
Sanjay Patel [Fri, 1 Nov 2019 13:25:08 +0000 (09:25 -0400)]
[SLP] avoid 'tmp' value name conflict with auto-generated CHECK script; NFC

The script uses 'TMP#' as its substitute for nameless values,
so if a test already contains 'tmp#' *named* values, then
there could be trouble. We should probably just fix the
script to avoid this problem going forward, but it's easy
enough to change a test too (and explicitly naming variables
'tmp' is always a sad choice).

4 years ago[SLP] avoid 'tmp' value name conflict with auto-generated CHECK script; NFC
Sanjay Patel [Fri, 1 Nov 2019 13:09:53 +0000 (09:09 -0400)]
[SLP] avoid 'tmp' value name conflict with auto-generated CHECK script; NFC

The script uses 'TMP#' as its substitute for nameless values,
so if a test already contains 'tmp#' *named* values, then
there could be trouble. We should probably just fix the
script to avoid this problem going forward, but it's easy
enough to change a test too (and explicitly naming variables
'tmp' is always a sad choice).

4 years agoUnmask dexter debuginfo tests on Darwin
Jeremy Morse [Fri, 1 Nov 2019 13:09:37 +0000 (13:09 +0000)]
Unmask dexter debuginfo tests on Darwin

These tests almost certainly work on Darwin anyway, I just wanted to
keep things in a fixed, working configuration, while pushing Dexter
up.

I've left Windows unsupported as the dexter command line will need further
adjustment to run dbgeng. This can be abstracted through the %dexter
substitution, but is a task for another time.

4 years ago[libcxx] Disable -Wconstant-evaluated for testsuite
Dávid Bolvanský [Fri, 1 Nov 2019 12:46:05 +0000 (13:46 +0100)]
[libcxx] Disable -Wconstant-evaluated for testsuite

Reviewers: EricWF

Subscribers: christof, ldionne, libcxx-commits

Tags: #libc

Differential Revision: https://reviews.llvm.org/D69708

4 years agoFix a brain-fail with debuginfo-tests/dexter internal tests
Jeremy Morse [Fri, 1 Nov 2019 12:29:42 +0000 (12:29 +0000)]
Fix a brain-fail with debuginfo-tests/dexter internal tests

I baked the "test" subcommand into the %dexter substituion, as that's
what all of the dexter tests use. However I forgot about the internal
tests for whether dexters features are working. Install a %dexter_base
command to allow those tests to call dexter.py directly, and un-xfail
the tests on darwin.

Update too the list of paths the unittests shouldn't try and cover, as it
tries to load dbgeng on unix machines. Ideally we wouldn't be using this
method of test discovery in the future.

4 years ago[MIPS GlobalISel] Improve reg bank handling in MipsInstructionSelector
Petar Avramovic [Fri, 1 Nov 2019 12:24:07 +0000 (13:24 +0100)]
[MIPS GlobalISel] Improve reg bank handling in MipsInstructionSelector

Introduce helper methods and refactor pieces of code related to
register banks in MipsInstructionSelector.
Add a few detailed asserts in order to get a better overview
of LLT, register bank combinations that are supported at the moment
and reduce need to look at other files.

Differential Revision: https://reviews.llvm.org/D69663

4 years ago[LegacyPM] Fix pass structure dumping
evgeny [Fri, 1 Nov 2019 11:43:51 +0000 (14:43 +0300)]
[LegacyPM] Fix pass structure dumping

If module pass uses on-demand function analyses then structure is being
displayed incorrectly because FunctionPassManagerImpl can't dump contained
FPPassManager instances.

Differential revision: https://reviews.llvm.org/D69315

4 years ago[NFC][llvm-readobj] Split getSectionIndexName function into two
James Henderson [Fri, 1 Nov 2019 10:27:00 +0000 (10:27 +0000)]
[NFC][llvm-readobj] Split getSectionIndexName function into two

getSectionIndexName was trying to fetch two things at once, which led to
a somewhat tricky to understand interface involving passing output
parameters in, and also made it hard to return Errors further up the
stack.

This change is in preparation for changing the error handling.

Additionally, update a related test now that yaml2obj supports
SHT_SYMTAB_SHNDX properly (see d3963051c490), and add missing LLVM-style
coverage for symbols with shndx SHN_XINDEX. This test (after fixing)
caught a mistake in my first attempt at this patch, hence I'm including
it as part of this patch.

Reviewed by: grimar, MaskRay

Differential Revision: https://reviews.llvm.org/D69670

4 years ago[NFC][llvm-readobj] Pull common code into a helper
James Henderson [Fri, 1 Nov 2019 10:16:40 +0000 (10:16 +0000)]
[NFC][llvm-readobj] Pull common code into a helper

This will make planned changes to this code easier to make.

Reviewed by: MaskRay, grimar

Differential Revision: https://reviews.llvm.org/D69669

4 years ago[AArch64][SVE] Implement several floating-point arithmetic intrinsics
Kerry McLaughlin [Fri, 1 Nov 2019 10:40:36 +0000 (10:40 +0000)]
[AArch64][SVE] Implement several floating-point arithmetic intrinsics

Summary:
Adds intrinsics for the following:
  - fabd, fadd, fsub & fsubr
  - fmul, fmulx, fdiv & fdivr
  - fmax, fmaxnm, fmin & fminnm
  - fscale & ftsmul

Reviewers: huntergr, sdesmalen, dancgr

Reviewed By: sdesmalen

Subscribers: tschuett, kristof.beyls, hiraditya, rkruppe, psnobl, cameron.mcinally, cfe-commits, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69657

4 years ago[GlobalISel] Match table opt: fix a bug in matching num of operands
Roman Tereshin [Thu, 31 Oct 2019 03:58:46 +0000 (20:58 -0700)]
[GlobalISel] Match table opt: fix a bug in matching num of operands

If there is a dag node with a variable number of operands that has at
least N operands (for some non-negative N), and multiple patterns with
that node with different number of operands, we would drop the number of
operands check in patterns with N operands, presumably because it's
guaranteed in such case that none of the per-operand checks will access
the operand list out-of-bounds.

Except semantically the check is about having exactly N operands, not at
least N operands, and a backend might rely on it to disambiguate
different patterns.

In this patch we change the condition on emitting the number of operands
check from "the instruction is not guaranteed to have at least as many
operands as are checked by the pattern being matched" to "the
instruction is not guaranteed to have a specific number of operands".

We're relying (still) on the rest of the CodeGenPatterns mechanics to
validate that the pattern itself doesn't try to access more operands
than there is in the instruction in cases when the instruction does have
fixed number of operands, and on the machine verifier to validate at
runtime that particular MIs like that satisfy the constraint as well.

Reviewers: dsanders, qcolombet

Reviewed By: qcolombet

Subscribers: arsenm, rovka, Petar.Avramovic, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69653

4 years ago[compiler-rt] [profile] Fix building for MinGW after d889d1efefe9f
Martin Storsjö [Fri, 1 Nov 2019 07:20:51 +0000 (09:20 +0200)]
[compiler-rt] [profile] Fix building for MinGW after d889d1efefe9f

This commit added use of a Windows API in InstrProfilingPort.h.
When _MSC_VER is defined (for MSVC), windows.h is already included
earlier in the same header (for atomics), but MinGW, the gcc
atomics builtins are used instead. Therefore explicitly include
windows.h here, where the API is used.

4 years agoAMDGPU: Add default denormal mode to MachineFunctionInfo
Matt Arsenault [Mon, 28 Oct 2019 06:38:52 +0000 (23:38 -0700)]
AMDGPU: Add default denormal mode to MachineFunctionInfo

The default FP mode should really be a property of a specific
function, and not a subtarget. Introduce the necessary fields to the
SIMachineFunctionInfo to help move towards this goal.

4 years ago[X86] Reland: Enable YMM memcmp with AVX1
David Zarzycki [Thu, 31 Oct 2019 10:30:53 +0000 (12:30 +0200)]
[X86] Reland: Enable YMM memcmp with AVX1

Update TargetTransformInfo to allow AVX1 to use YMM registers for memcmp.

This is a follow up to D68632 which enabled XOR compares which made this possible.

This also updates the memcmp-optsize.ll test unlike the first patch.

https://reviews.llvm.org/D69658

4 years ago[utils] Reflow asm check generation to tolerate blank lines
Simon Atanasyan [Wed, 30 Oct 2019 16:52:16 +0000 (19:52 +0300)]
[utils] Reflow asm check generation to tolerate blank lines

This change introduces two fixes. The second fix allows to generate
a test to check the first fix.

- Output `CHECK-EMPTY` prefix for an empty line in ASM output. Before that
  fix `update_llc_test_checks.py` incorrectly emits `CHECK-NEXT: <space>`
  prefix.
- Fix the `ASM_FUNCTION_MIPS_RE` regex to stop on a real function
  epilogue not on an inline assembler prologue and include inline
  assembler code into a test.

Differential Revision: https://reviews.llvm.org/D47192

4 years agoDAG: Add DAG argument to isFPExtFoldable
Matt Arsenault [Mon, 28 Oct 2019 23:06:34 +0000 (16:06 -0700)]
DAG: Add DAG argument to isFPExtFoldable

For AMDGPU this is dependent on the FP mode, which should eventually
not be a property of the subtarget.

4 years ago[WebAssembly] SIMD integer min and max instructions
Thomas Lively [Thu, 31 Oct 2019 23:49:47 +0000 (16:49 -0700)]
[WebAssembly] SIMD integer min and max instructions

Summary:
Introduces a clang builtins and LLVM intrinsics representing integer
min/max instructions. These instructions have not been merged to the
SIMD spec proposal yet, so they are currently opt-in only via builtins
and not produced by general pattern matching. If these instructions
are accepted into the spec proposal the builtins and intrinsics will
be replaced with normal pattern matching.

Defined in https://github.com/WebAssembly/simd/pull/27.

Reviewers: aheejin

Reviewed By: aheejin

Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, cfe-commits, llvm-commits

Tags: #clang, #llvm

Differential Revision: https://reviews.llvm.org/D69696

4 years agoFix update_mir_test_checks after 3598b8100
Matt Arsenault [Fri, 1 Nov 2019 03:08:28 +0000 (20:08 -0700)]
Fix update_mir_test_checks after 3598b8100

4 years agoclang: Fix assert on void pointer arithmetic with address_space
Matt Arsenault [Thu, 31 Oct 2019 15:41:37 +0000 (08:41 -0700)]
clang: Fix assert on void pointer arithmetic with address_space

This attempted to always use the default address space void pointer
type instead of preserving the source address space.

4 years agoReland "[WebAssembly] Handle multiple loads of splatted loads"
Thomas Lively [Fri, 1 Nov 2019 03:01:02 +0000 (20:01 -0700)]
Reland "[WebAssembly] Handle multiple loads of splatted loads"

This reverts commit 92a25fbf11da51c8e3573b81a877d3b226990c07 and fixes
the ambiguous method call that was causing build failures.

4 years ago[clang][driver] Add ProfileData to LLVM_LINK_COMPONENTS
Heejin Ahn [Fri, 1 Nov 2019 02:48:13 +0000 (19:48 -0700)]
[clang][driver] Add ProfileData to LLVM_LINK_COMPONENTS

Summary:
After D68351 we need this to make builds with `-DBUILD_SHARED_LIB=ON`
work.

Reviewers: tlively

Subscribers: mgorny, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D69699

4 years ago[CodeGen] Fix invalid llvm.linker.options about pragma detect_mismatch
Yaxun (Sam) Liu [Thu, 31 Oct 2019 19:47:21 +0000 (15:47 -0400)]
[CodeGen] Fix invalid llvm.linker.options about pragma detect_mismatch

When a target does not support pragma detect_mismatch, an llvm.linker.options
metadata with an empty entry is created, which causes diagnostic in backend
since backend expects name/value pair in llvm.linker.options entries.

This patch fixes that.

Differential Revision: https://reviews.llvm.org/D69678

4 years ago[nfc][omptarget] Use builtin var abstraction. Second pass at D69476
JonChesterfield [Fri, 1 Nov 2019 02:21:12 +0000 (02:21 +0000)]
[nfc][omptarget] Use builtin var abstraction. Second pass at D69476

Summary:
[nfc][omptarget] Use builtin var abstraction. Second pass at D69476

Use the wrappers in support.h for cuda builtin variables at all call sites.
Localises use of cuda and removes WARPSIZE==32 assumption in debug.h.

Reviewers: ABataev, jdoerfert, grokos

Reviewed By: jdoerfert

Subscribers: openmp-commits

Tags: #openmp

Differential Revision: https://reviews.llvm.org/D69693

4 years ago[profile] Fifth speculative fix for Android after D68351
Vedant Kumar [Fri, 1 Nov 2019 00:03:51 +0000 (17:03 -0700)]
[profile] Fifth speculative fix for Android after D68351

Use the printf macros from inttypes.h to sidestep -Wformat issues:

/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/compiler-rt/lib/profile/InstrProfilingFile.c:425:14: error: format specifies type 'long long' but the argument has type 'off_t' (aka 'long') [-Werror,-Wformat]
             CurrentFileOffset, PageSize);
             ^~~~~~~~~~~~~~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/compiler-rt/lib/profile/InstrProfilingPort.h:114:50: note: expanded from macro 'PROF_ERR'
  fprintf(stderr, "LLVM Profile Error: " Format, __VA_ARGS__);
                                         ~~~~~~  ^~~~~~~~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/compiler-rt/lib/profile/InstrProfilingFile.c:461:41: error: format specifies type 'unsigned long long' but the argument has type 'uint64_t' (aka 'unsigned long') [-Werror,-Wformat]
        strerror(errno), CountersBegin, PageAlignedCountersLength, Fileno,
                                        ^~~~~~~~~~~~~~~~~~~~~~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/compiler-rt/lib/profile/InstrProfilingPort.h:114:50: note: expanded from macro 'PROF_ERR'
  fprintf(stderr, "LLVM Profile Error: " Format, __VA_ARGS__);
                                         ~~~~~~  ^~~~~~~~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/compiler-rt/lib/profile/InstrProfilingFile.c:462:9: error: format specifies type 'unsigned long long' but the argument has type 'uint64_t' (aka 'unsigned long') [-Werror,-Wformat]
        FileOffsetToCounters);
        ^~~~~~~~~~~~~~~~~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/compiler-rt/lib/profile/InstrProfilingPort.h:114:50: note: expanded from macro 'PROF_ERR'
  fprintf(stderr, "LLVM Profile Error: " Format, __VA_ARGS__);

4 years agoRevert "[WebAssembly] Handle multiple loads of splatted loads"
Vlad Tsyrklevich [Thu, 31 Oct 2019 23:50:36 +0000 (16:50 -0700)]
Revert "[WebAssembly] Handle multiple loads of splatted loads"

This reverts commit 2ab1b8c1ec452fb743f6cc5051e75a01039cabfe, it is
causing build failures on numerous bots, including
sanitizer-x86_64-linux-bootstrap-ubsan. My previous revert was for the
wrong commit.

4 years ago[profile] Fourth fix for toolchains without zlib after D68351
Vedant Kumar [Thu, 31 Oct 2019 23:45:39 +0000 (16:45 -0700)]
[profile] Fourth fix for toolchains without zlib after D68351

Fix llvm/test/tools/llvm-profdata/nocompress.test, which I missed when
authoring D68351 because my toolchain is configured with zlib support.

4 years agoRevert "[WebAssembly] Expand setcc of v2i64"
Vlad Tsyrklevich [Thu, 31 Oct 2019 23:32:38 +0000 (16:32 -0700)]
Revert "[WebAssembly] Expand setcc of v2i64"

This reverts commit 11850a6305c5778b180243eb06aefe86762dd4ce, it was
causing build failures on numerous bots, including
sanitizer-x86_64-linux-bootstrap-ubsan.

4 years ago[profile] Third speculative fix for Windows after D68351
Vedant Kumar [Thu, 31 Oct 2019 23:35:57 +0000 (16:35 -0700)]
[profile] Third speculative fix for Windows after D68351

_putenv on Windows takes 1 argument, whereas setenv elsewhere takes 3.
Just treat the two platforms differently.

http://lab.llvm.org:8011/builders/sanitizer-windows/builds/53547

4 years ago[profile] Second speculative fix for Windows
Vedant Kumar [Thu, 31 Oct 2019 23:25:24 +0000 (16:25 -0700)]
[profile] Second speculative fix for Windows

VLAs in C appear to not work on Windows, so use COMPILER_RT_ALLOCA:

C:\b\slave\sanitizer-windows\llvm-project\compiler-rt\lib\profile\InstrProfilingWriter.c(264): error C2057: expected constant expression
C:\b\slave\sanitizer-windows\llvm-project\compiler-rt\lib\profile\InstrProfilingWriter.c(264): error C2466: cannot allocate an array of constant size 0
C:\b\slave\sanitizer-windows\llvm-project\compiler-rt\lib\profile\InstrProfilingWriter.c(264): error C2133: 'Zeroes': unknown size

4 years ago[profile] Speculative fix for Windows after D68351
Vedant Kumar [Thu, 31 Oct 2019 23:22:07 +0000 (16:22 -0700)]
[profile] Speculative fix for Windows after D68351

setenv() appears to not be available on Windows:

http://lab.llvm.org:8011/builders/sanitizer-windows/builds/53545/steps/stage%201%20build/logs/stdio

4 years agogn build: Merge e72e59e9028
LLVM GN Syncbot [Thu, 31 Oct 2019 23:15:09 +0000 (23:15 +0000)]
gn build: Merge e72e59e9028

4 years agoRevert "[X86] add mayRaiseFPException flag and FPCW registers for X87 instructions"
Nico Weber [Thu, 31 Oct 2019 23:14:42 +0000 (19:14 -0400)]
Revert "[X86] add mayRaiseFPException flag and FPCW registers for X87 instructions"

This reverts commit a678677da498a45f59c16ee74fea438e34a801ce.
It broke CodeGen/ms-inline-asm.c on most bots.

4 years ago[profile] Add a mode to continuously sync counter updates to a file
Vedant Kumar [Thu, 19 Sep 2019 18:56:43 +0000 (11:56 -0700)]
[profile] Add a mode to continuously sync counter updates to a file

Add support for continuously syncing profile counter updates to a file.

The motivation for this is that programs do not always exit cleanly. On
iOS, for example, programs are usually killed via a signal from the OS.
Running atexit() handlers after catching a signal is unreliable, so some
method for progressively writing out profile data is necessary.

The approach taken here is to mmap() the `__llvm_prf_cnts` section onto
a raw profile. To do this, the linker must page-align the counter and
data sections, and the runtime must ensure that counters are mapped to a
page-aligned offset within a raw profile.

Continuous mode is (for the moment) incompatible with the online merging
mode. This limitation is lifted in https://reviews.llvm.org/D69586.

Continuous mode is also (for the moment) incompatible with value
profiling, as I'm not sure whether there is interest in this and the
implementation may be tricky.

As I have not been able to test extensively on non-Darwin platforms,
only Darwin support is included for the moment. However, continuous mode
may "just work" without modification on Linux and some UNIX-likes. AIUI
the default value for the GNU linker's `--section-alignment` flag is set
to the page size on many systems. This appears to be true for LLD as
well, as its `no_nmagic` option is on by default. Continuous mode will
not "just work" on Fuchsia or Windows, as it's not possible to mmap() a
section on these platforms. There is a proposal to add a layer of
indirection to the profile instrumentation to support these platforms.

rdar://54210980

Differential Revision: https://reviews.llvm.org/D68351

4 years ago[profile] Clean up stale raw profiles in instrprof-write-file.c
Vedant Kumar [Wed, 2 Oct 2019 01:14:54 +0000 (18:14 -0700)]
[profile] Clean up stale raw profiles in instrprof-write-file.c

4 years agoFix a few typos in SourceLevelDebugging.rst
Adrian Prantl [Thu, 31 Oct 2019 23:03:18 +0000 (16:03 -0700)]
Fix a few typos in SourceLevelDebugging.rst

4 years ago[Remarks] Fix error message check in unit test
Francis Visoiu Mistrih [Thu, 31 Oct 2019 22:50:00 +0000 (15:50 -0700)]
[Remarks] Fix error message check in unit test

Always compare using lowercase to support multiple platforms.

4 years agogn build: Add support for cross-compiling the builtins and profile runtimes for Andro...
Peter Collingbourne [Thu, 31 Oct 2019 22:15:54 +0000 (15:15 -0700)]
gn build: Add support for cross-compiling the builtins and profile runtimes for Android aarch64 and arm.

Differential Revision: https://reviews.llvm.org/D69681

4 years ago[Remarks] Add support for linking remarks
Francis Visoiu Mistrih [Tue, 10 Sep 2019 22:48:55 +0000 (15:48 -0700)]
[Remarks] Add support for linking remarks

Remarks are usually emitted per-TU, and for generating a standalone
remark file that can be shipped with the linked binary we need some kind
of tool to merge everything together.

The remarks::RemarkLinker class takes care of this and:

* Deduplicates remarks
* Filters remarks with no debug location
* Merges string tables from all the entries

As an output, it provides an iterator range that can be used to
serialize the remarks to a file.

Differential Revision: https://reviews.llvm.org/D69141

4 years agoQuote path to Python 3 executable in case it contains spaces
Reid Kleckner [Thu, 31 Oct 2019 22:23:45 +0000 (15:23 -0700)]
Quote path to Python 3 executable in case it contains spaces

4 years agoFix undefined variable bug in debuginfotests lit.cfg.py
Reid Kleckner [Thu, 31 Oct 2019 22:18:54 +0000 (15:18 -0700)]
Fix undefined variable bug in debuginfotests lit.cfg.py

4 years ago[LV] Add test case that was supposed to go with D67948
Craig Topper [Thu, 31 Oct 2019 22:10:39 +0000 (15:10 -0700)]
[LV] Add test case that was supposed to go with D67948

I forgot to git add it when I committed for Evgeniy.

4 years ago[X86] add mayRaiseFPException flag and FPCW registers for X87 instructions
Craig Topper [Thu, 31 Oct 2019 21:48:38 +0000 (14:48 -0700)]
[X86] add mayRaiseFPException flag and FPCW registers for X87 instructions

This patch adds flag "mayRaiseFPException" , FPCW and FPSW for X87 instructions which could raise
float exception.

Patch by LiuChen. With a couple small fixes from me.

Differential Revision: https://reviews.llvm.org/D68854

4 years ago[WebAssembly] Handle multiple loads of splatted loads
Thomas Lively [Wed, 30 Oct 2019 20:22:13 +0000 (13:22 -0700)]
[WebAssembly] Handle multiple loads of splatted loads

Summary:
Fixes an ISel failure when a splatted load is used more than once. The
failure was due to the hacks we were doing in ISel lowering to
preserve the original load as the operand of a LOAD_SPLAT node. The
fix is to properly lower the splatted use of the load to a separate
LOAD_SPLAT node.

Reviewers: aheejin

Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69640

4 years ago[mca] Fix test case (NFC)
Evandro Menezes [Thu, 31 Oct 2019 21:42:55 +0000 (16:42 -0500)]
[mca] Fix test case (NFC)

Fix test case for Darwin builds.

4 years ago[scudo][standalone] Fix Secondary bug w/ freelist
Kostya Kortchinsky [Thu, 31 Oct 2019 17:31:49 +0000 (10:31 -0700)]
[scudo][standalone] Fix Secondary bug w/ freelist

Summary:
cferris@ found an issue due to the new Secondary free list behavior
and unfortunately it's completely my fault. The issue is twofold:

- I lost track of the (major) fact that the Combined assumes that
  all chunks returned by the Secondary are zero'd out apprioriately
  when dealing with `ZeroContents`. With the introduction of the
  freelist, it's no longer the case as there can be a small portion
  of memory between the header and the next page boundary that is
  left untouched (the rest is zero'd via release). So the next time
  that block is returned, it's not fully zero'd out.
- There was no test that would exercise that behavior :(

There are several ways to fix this, the one I chose makes the most
sense to me: we pass `ZeroContents` to the Secondary's `allocate`
and it zero's out the block if requested and it's coming from the
freelist. The prevents an extraneous `memset` in case the block
comes from `map`. Another possbility could have been to `memset`
in `deallocate`, but it's probably overzealous as all secondary
blocks don't need to be zero'd out.

Add a test that would have found the issue prior to fix.

Reviewers: morehouse, hctim, cferris, pcc, eugenis, vitalybuka

Subscribers: #sanitizers, llvm-commits

Tags: #sanitizers, #llvm

Differential Revision: https://reviews.llvm.org/D69675

4 years agoRefactor getDeclAtPosition() to use SelectionTree + targetDecl()
Nathan Ridge [Thu, 17 Oct 2019 22:48:39 +0000 (18:48 -0400)]
Refactor getDeclAtPosition() to use SelectionTree + targetDecl()

Summary: This fixes issue #163, among other improvements to go-to-definition.

Reviewers: sammccall

Subscribers: jkorous, mgrang, arphaman, kadircet, usaxena95, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D69237