platform/upstream/llvm.git
3 years ago[OpenMP] Move memory manager to plugin and make it a common interface
Shilei Tian [Tue, 12 Jan 2021 02:33:25 +0000 (21:33 -0500)]
[OpenMP] Move memory manager to plugin and make it a common interface

The lifetime of `libomptarget` and its opened plugins are not aligned
and it's hard for `libomptarget` to determine when the plugins are destroyed.
As a result, some issues (see D94256 for details) occur on some platforms.
Actually, if we take target memory as target resources, same as other resources,
such as CUDA streams, in each plugin, then the memory manager should also be in
the plugin. Also considering some platforms may want to opt out the feature, it
makes sense to move the memory manager to plugin, make it a common interface, and
let plguin developers determine whether they need it. This is what this patch does.
CUDA plugin is taken as example to show how to integrate it. In this way, we can
also get a bonus that different thresholds can be set for different platforms.

Reviewed By: jdoerfert, JonChesterfield

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

3 years ago[NFC][LICM] Minor improvements to debug output
Quentin Colombet [Tue, 12 Jan 2021 00:31:40 +0000 (16:31 -0800)]
[NFC][LICM] Minor improvements to debug output

Added a utility function in Value class to print block name and use
block labels for unnamed blocks.
Changed LICM to call this function in its debug output.

Patch by Xiaoqing Wu <xiaoqing_wu@apple.com>

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

3 years ago[WebAssembly] Ensure terminate pads are a single BB
Heejin Ahn [Sun, 27 Dec 2020 11:02:22 +0000 (03:02 -0800)]
[WebAssembly] Ensure terminate pads are a single BB

This ensures every single terminate pad is a single BB in the form of:
```
%exn = catch $__cpp_exception
call @__clang_call_terminate(%exn)
unreachable
```

This is a preparation for HandleEHTerminatePads pass, which will be
added in a later CL and will run after CFGStackify. That pass duplicates
terminate pads with a `catch_all` instruction, and duplicating it
becomes simpler if we can ensure every terminate pad is a single BB.

Reviewed By: dschuff, tlively

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

3 years ago[VE][compiler-rt] Add VE as a target of crt
Kazushi (Jam) Marukawa [Fri, 4 Dec 2020 23:41:46 +0000 (08:41 +0900)]
[VE][compiler-rt] Add VE as a target of crt

SX Aurora VE is an experimental target.  We upstreamed many part of
ported llvm and clang.  In order to continue this move, we need to
support libraries next, then we need to show the ability of llvm for
VE through test cases.  As a first step for that, we need to use
crt in compiler-rt.  VE has it's own crt but they are a part of
proprietary compiler.  So, we want to use crt in compiler-rt as an
alternative.

This patch enables VE as a candidate of crt in compiler-rt.

Reviewed By: phosek, compnerd

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

3 years ago[JITLink] Add a new PostAllocationPasses list.
Lang Hames [Mon, 11 Jan 2021 09:00:45 +0000 (20:00 +1100)]
[JITLink] Add a new PostAllocationPasses list.

Passes in the new PostAllocationPasses list will run immediately after memory
allocation and address assignment for defined symbols, and before
JITLinkContext::notifyResolved is called. These passes can set up state
associated with the addresses of defined symbols before any query for these
addresses completes.

3 years ago[MC] Make getEHFrameSection const like every other getter (NFC)
Jonas Devlieghere [Tue, 12 Jan 2021 00:56:02 +0000 (16:56 -0800)]
[MC] Make getEHFrameSection const like every other getter (NFC)

3 years ago[MLIR][Linalg] Refactor transforms to use linalg::getDynOperands helper
Rob Suderman [Fri, 8 Jan 2021 22:03:12 +0000 (14:03 -0800)]
[MLIR][Linalg] Refactor transforms to use linalg::getDynOperands helper

getDynOperands behavior is commonly used in a number of passes. Refactored to
use a helper function and avoid code reuse.

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

3 years ago[github] Move repo lockdown config into llvm-project repo
Tom Stellard [Tue, 12 Jan 2021 00:20:08 +0000 (16:20 -0800)]
[github] Move repo lockdown config into llvm-project repo

We were storing this in the .github repo and moving this into llvm-project
will allow us to delete the .github repo.

3 years ago[clang] Fix message text for `-Wpointer-sign` to account for plain char
Hubert Tong [Mon, 11 Jan 2021 23:36:28 +0000 (18:36 -0500)]
[clang] Fix message text for `-Wpointer-sign` to account for plain char

The `-Wpointer-sign` warning text is inappropriate for describing the
incompatible pointer conversion between plain `char` and explicitly
`signed`/`unsigned` `char` (whichever plain `char` has the same range
as) and vice versa.

Specifically, in part, it reads "converts between pointers to integer
types with different sign". This patch changes that portion to read
instead as "converts between pointers to integer types where one is of
the unique plain 'char' type and the other is not" when one of the types
is plain `char`.

C17 subclause 6.5.16.1 indicates that the conversions resulting in
`-Wpointer-sign` warnings in assignment-like contexts are constraint
violations. This means that strict conformance requires a diagnostic for
the case where the message text is wrong before this patch. The lack of
an even more specialized warning group is consistent with GCC.

Reviewed By: aaron.ballman

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

3 years agoNFC: Pre-commit test: -Wpointer-sign with plain char to [un]signed char
Hubert Tong [Mon, 11 Jan 2021 23:35:54 +0000 (18:35 -0500)]
NFC: Pre-commit test: -Wpointer-sign with plain char to [un]signed char

Add tests with bad message text for `-Wpointer-sign` and run them with
both signed and unsigned versions of plain `char`.

3 years ago[RISCV] Define the vfclass RVV intrinsics
Evandro Menezes [Thu, 31 Dec 2020 03:51:41 +0000 (21:51 -0600)]
[RISCV] Define the vfclass RVV intrinsics

Define the `vfclass` IR intrinsics for the respective V instructions.

Authored-by: Roger Ferrer Ibanez <rofirrim@gmail.com>
Co-Authored-by: Evandro Menezes <evandro.menezes@sifive.com>
Differential Revision: https://reviews.llvm.org/D94356

3 years agoADT: Fix pointer comparison UB in SmallVector
Duncan P. N. Exon Smith [Wed, 23 Dec 2020 21:17:45 +0000 (13:17 -0800)]
ADT: Fix pointer comparison UB in SmallVector

The standard requires comparisons of pointers to unrelated storage to
use `std::less`. Split out some helpers that do that and update all the
code that was comparing using `<` and friends (mostly assertions).

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

3 years ago[SimplifyCFGPass] iterativelySimplifyCFG(): support lazy DomTreeUpdater
Roman Lebedev [Mon, 11 Jan 2021 21:36:18 +0000 (00:36 +0300)]
[SimplifyCFGPass] iterativelySimplifyCFG(): support lazy DomTreeUpdater

This boils down to how we deal with early-increment iterator
over function's basic blocks: not only we need to early-increment,
after that we also need to skip all the blocks
that are scheduled for removal, as per DomTreeUpdater.

3 years ago[SimplifyCFGPass] mergeEmptyReturnBlocks(): skip blocks scheduled for removal as...
Roman Lebedev [Mon, 11 Jan 2021 21:35:15 +0000 (00:35 +0300)]
[SimplifyCFGPass] mergeEmptyReturnBlocks(): skip blocks scheduled for removal as per DomTreeUpdater

Thus supporting lazy DomTreeUpdater mode,
where the domtree updates (and thus block removals)
aren't applied immediately, but are delayed
until last possible moment.

3 years ago[NFCI][Utils/Local] removeUnreachableBlocks(): cleanup support for lazy DomTreeUpdater
Roman Lebedev [Mon, 11 Jan 2021 21:29:55 +0000 (00:29 +0300)]
[NFCI][Utils/Local] removeUnreachableBlocks(): cleanup support for lazy DomTreeUpdater

When DomTreeUpdater is in lazy update mode, the blocks
that were scheduled to be removed, won't be removed
until the updates are flushed, e.g. by asking
DomTreeUpdater for a up-to-date DomTree.

From the function's current code, it is pretty evident
that the support for the lazy mode is an afterthought,
see e.g. how we roll-back NumRemoved statistic..

So instead of considering all the unreachable blocks
as the blocks-to-be-removed, simply additionally skip
all the blocks that are already scheduled to be removed

3 years ago[SimplifyCFG] FoldValueComparisonIntoPredecessors(): don't insert a DomTree edge...
Roman Lebedev [Mon, 11 Jan 2021 21:21:25 +0000 (00:21 +0300)]
[SimplifyCFG] FoldValueComparisonIntoPredecessors(): don't insert a DomTree edge if it already exists

When we are adding edges to the terminator and potentially turning it
into a switch (if it wasn't already), it is possible that the
case we're adding will share it's destination with one of the
preexisting cases, in which case there is no domtree edge to add.

Indeed, this change does not have a test coverage change.
This failure has been exposed in an existing test coverage
by a follow-up patch that switches to lazy domtreeupdater mode,
and removes domtree verification from
SimplifyCFGOpt::simplifyOnce()/SimplifyCFGOpt::run(),
IOW it does not appear feasible to add dedicated test coverage here.

3 years ago[SimplifyCFG] SimplifyBranchOnICmpChain(): don't insert a DomTree edge that already...
Roman Lebedev [Mon, 11 Jan 2021 21:18:23 +0000 (00:18 +0300)]
[SimplifyCFG] SimplifyBranchOnICmpChain(): don't insert a DomTree edge that already exists

BB was already always branching to EdgeBB, there is no edge to add.

Indeed, this change does not have a test coverage change.
This failure has been exposed in an existing test coverage
by a follow-up patch that switches to lazy domtreeupdater mode,
and removes domtree verification from
SimplifyCFGOpt::simplifyOnce()/SimplifyCFGOpt::run(),
IOW it does not appear feasible to add dedicated test coverage here.

3 years ago[SimplifyCFG] SwitchToLookupTable(): don't insert a DomTree edge that already exists
Roman Lebedev [Mon, 11 Jan 2021 13:01:57 +0000 (16:01 +0300)]
[SimplifyCFG] SwitchToLookupTable(): don't insert a DomTree edge that already exists

SI is the terminator of BB, so the edge we are adding obviously already existed.

Indeed, this change does not have a test coverage change.
This failure has been exposed in an existing test coverage
by a follow-up patch that switches to lazy domtreeupdater mode,
and removes domtree verification from
SimplifyCFGOpt::simplifyOnce()/SimplifyCFGOpt::run(),
IOW it does not appear feasible to add dedicated test coverage here.

3 years ago[RISCV] Use vmv.v.i vd, 0 instead of vmv.v.x vd, x0 for llvm.riscv.vfmv.v.f with 0.0
Craig Topper [Mon, 11 Jan 2021 23:03:29 +0000 (15:03 -0800)]
[RISCV] Use vmv.v.i vd, 0 instead of vmv.v.x vd, x0 for llvm.riscv.vfmv.v.f with 0.0

This matches what we use for integer 0. It's also consistent with
the scalar 'mv' pseudo that uses addi rather than add with x0.

3 years agoUpdate syntax for tensor and memref types to match parser.
Richard Uhler [Mon, 11 Jan 2021 22:57:05 +0000 (22:57 +0000)]
Update syntax for tensor and memref types to match parser.

Based on the comments in lib/Parser/TypeParser.cpp on the
parseMemRefType and parseTensorType functions.

Reviewed By: mehdi_amini

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

3 years agoclang: Always pass PowerPC endian information to GNU as
Nathan Chancellor [Mon, 11 Jan 2021 22:39:01 +0000 (14:39 -0800)]
clang: Always pass PowerPC endian information to GNU as

When building a 64-bit big endian PowerPC Linux kernel with a 64-bit
little endian PowerPC target, the 32-bit vDSO errors:

```
$ make ARCH=powerpc CC=clang CROSS_COMPILE=powerpc64le-linux-gnu- \
       pseries_defconfig arch/powerpc/kernel/vdso32/
ld.lld: error: arch/powerpc/kernel/vdso32/sigtramp.o is incompatible with elf32-powerpc
ld.lld: error: arch/powerpc/kernel/vdso32/gettimeofday.o is incompatible with elf32-powerpc
ld.lld: error: arch/powerpc/kernel/vdso32/datapage.o is incompatible with elf32-powerpc
ld.lld: error: arch/powerpc/kernel/vdso32/cacheflush.o is incompatible with elf32-powerpc
ld.lld: error: arch/powerpc/kernel/vdso32/note.o is incompatible with elf32-powerpc
ld.lld: error: arch/powerpc/kernel/vdso32/getcpu.o is incompatible with elf32-powerpc
ld.lld: error: arch/powerpc/kernel/vdso32/vgettimeofday.o is incompatible with elf32-powerpc
...
```

This happens because the endian information is missing from the call to
the assembler, even though it was explicitly passed to clang. See the
below example.

```
$ echo | clang --target=powerpc64le-linux-gnu \
               --prefix=/usr/bin/powerpc64le-linux-gnu- \
               -no-integrated-as -m32 -mbig-endian -### -x c -c -
 ".../clang-12" "-cc1" "-triple" "powerpc-unknown-linux-gnu" ...
...
  "/usr/bin/powerpc64le-linux-gnu-as" "-a32" "-mppc" "-many" "-o" "-.o" "/tmp/--e69e28.s"
```

clang sets the right target with -m32 and -mbig-endian but -mbig-endian
does not make it to the assembler, resulting in a 32-bit little endian
binary. This differs from the little endian targets, which always pass
-mlittle-endian.

```
$ echo | clang --target=powerpc64-linux-gnu \
               --prefix=/usr/bin/powerpc64-linux-gnu- \
               -no-integrated-as -m32 -mlittle-endian -### -x c -c -
 ".../clang-12" "-cc1" "-triple" "powerpcle-unknown-linux-gnu" ...
...
 "/usr/bin/powerpc64-linux-gnu-as" "-a32" "-mppc" "-mlittle-endian" "-many" "-o" "-.o" "/tmp/--405dbd.s"
```

Do the same thing for the big endian targets so that there is no more
error. This matches GCC's behavior, where -mbig and -mlittle are always
passed along to GNU as.

```
$ echo | powerpc64-linux-gcc -### -x c -c -
...
.../powerpc64-linux/bin/as -a64 -mpower4 -many -mbig -o -.o /tmp/ccVn7NAm.s
...

$ echo | powerpc64le-linux-gcc -### -x c -c -
...
.../powerpc64le-linux/bin/as -a64 -mpower8 -many -mlittle -o -.o /tmp/ccPN9ato.s
...
```

Reviewed By: nickdesaulniers, MaskRay

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

3 years ago[RISCV] Add scalable vector vselect ISel patterns
Fraser Cormack [Fri, 8 Jan 2021 11:47:12 +0000 (11:47 +0000)]
[RISCV] Add scalable vector vselect ISel patterns

Reviewed By: craig.topper

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

3 years ago[OpenMP] Take elf_common.c as a interface library
Shilei Tian [Mon, 11 Jan 2021 22:34:17 +0000 (17:34 -0500)]
[OpenMP] Take elf_common.c as a interface library

For now `elf_common.c` is taken as a common part included into
different plugin implementations directly via
`#include "../../common/elf_common.c"`, which is not a best practice. Since it
is simple enough such that we don't need to create a real library for it, we just
take it as a interface library so that other targets can link it directly. Another
advantage of this method is, we don't need to add the folder into header search
path which can potentially pollute the search path.

VE and AMD platforms have not been tested because I don't have target machines.

Reviewed By: JonChesterfield

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

3 years agoRename debug linkage name with -funique-internal-linkage-names
Hongtao Yu [Sat, 9 Jan 2021 06:57:03 +0000 (22:57 -0800)]
Rename debug linkage name with -funique-internal-linkage-names

Functions that are renamed under -funique-internal-linkage-names have their debug linkage name updated as well.

Reviewed By: dblaikie

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

3 years agoCGDebugInfo: Delete unused parameters
Fangrui Song [Mon, 11 Jan 2021 21:39:03 +0000 (13:39 -0800)]
CGDebugInfo: Delete unused parameters

3 years ago[SCCP] Fix misclassified conditions in test (NFC)
Nikita Popov [Mon, 11 Jan 2021 21:32:59 +0000 (22:32 +0100)]
[SCCP] Fix misclassified conditions in test (NFC)

3 years ago[PredicateInfo] Add test for one unknown condition in and/or (NFC)
Nikita Popov [Mon, 11 Jan 2021 19:42:47 +0000 (20:42 +0100)]
[PredicateInfo] Add test for one unknown condition in and/or (NFC)

Test the case where one part of and/or is an icmp, while the other
one is an arbitrary value.

3 years ago[mlir][vector] verify memref of vector memory ops
Aart Bik [Mon, 11 Jan 2021 19:34:10 +0000 (11:34 -0800)]
[mlir][vector] verify memref of vector memory ops

This ensures the memref base + indices expression is well-formed

Reviewed By: ThomasRaoux, ftynse

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

3 years ago[RISCV] Add scalable vector fadd/fsub/fmul/fdiv ISel patterns
Fraser Cormack [Mon, 11 Jan 2021 14:02:35 +0000 (14:02 +0000)]
[RISCV] Add scalable vector fadd/fsub/fmul/fdiv ISel patterns

Original patch by @rogfer01.

This patch adds ISel patterns for the above operations to the
corresponding vector/vector and vector/scalar RVV instructions, as well
as extra patterns to match operand-swapped scalar/vector vfrsub and
vfrdiv.

Authored-by: Roger Ferrer Ibanez <rofirrim@gmail.com>
Co-Authored-by: Fraser Cormack <fraser@codeplay.com>
Reviewed By: craig.topper

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

3 years agoAdd an assert to CGDebugInfo::getTypeOrNull
Fangrui Song [Mon, 11 Jan 2021 21:25:20 +0000 (13:25 -0800)]
Add an assert to CGDebugInfo::getTypeOrNull

3 years ago[c++20] Don't consider string literal operator templates for numeric
Richard Smith [Mon, 11 Jan 2021 21:12:43 +0000 (13:12 -0800)]
[c++20] Don't consider string literal operator templates for numeric
literals.

A literal interpretation of the standard wording allows this, but it was
never intended that string literal operator templates would be used for
anything other than user-defined string literals.

3 years ago[libc][NFC] Make __support/common.h an in tree header.
Siva Chandra Reddy [Mon, 11 Jan 2021 20:35:34 +0000 (12:35 -0800)]
[libc][NFC] Make __support/common.h an in tree header.

It was previously a generated header. It can easily converted to a
generated header if required in future.

Reviewed By: michaelrj

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

3 years ago[GlobalISel] Map extractelt to G_EXTRACT_VECTOR_ELT
Bjorn Pettersson [Mon, 11 Jan 2021 15:19:01 +0000 (16:19 +0100)]
[GlobalISel] Map extractelt to G_EXTRACT_VECTOR_ELT

Before this patch there was generic mapping from vector_extract
to G_EXTRACT_VECTOR_ELT added in SelectionDAGCompat.td. That
mapping is now replaced by a mapping from extractelt instead.

The reasoning is that vector_extract is marked as deprecated,
so it is assumed that a majority of targets will use extractelt
and not vector_extract (and that the long term solution for all
targets would be to use extractelt).

Targets like AArch64 that still use vector_extract can add an
additional mapping from the deprecated vector_extract as target
specific tablegen definitions. Such a mapping is added for AArch64
in this patch to avoid breaking tests.

When adding the extractelt => G_EXTRACT_VECTOR_ELT mapping we
triggered some new code paths in GlobalISelEmitter, ending up in
an assert when trying to import a pattern containing EXTRACT_SUBREG
for ARM. Therefore this patch also adds a "failedImport" warning
for that situation (instead of hitting the assert).

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

3 years ago[InstCombine] reduce icmp(ashr X, C1), C2 to sign-bit test
Sanjay Patel [Mon, 11 Jan 2021 20:36:22 +0000 (15:36 -0500)]
[InstCombine] reduce icmp(ashr X, C1), C2 to sign-bit test

This is a more basic pattern that we should handle before trying to solve:
https://llvm.org/PR48640

There might be a better way to think about this because the pre-condition
that I came up with (number of sign bits in the compare constant) misses a
potential transform for each of ugt and ult as commented on in the test file.

Tried to model this is in Alive:
https://rise4fun.com/Alive/juX1
...but I couldn't get the ComputeNumSignBits() pre-condition to work as
expected, so replaced with leading 0/1 preconditions instead.

  Name: ugt
  Pre: countLeadingZeros(C2) <= C1 && countLeadingOnes(C2) <= C1
  %a = ashr %x, C1
  %r = icmp ugt i8 %a, C2
    =>
  %r = icmp slt i8 %x, 0

  Name: ult
  Pre: countLeadingZeros(C2) <= C1 && countLeadingOnes(C2) <= C1
  %a = ashr %x, C1
  %r = icmp ult i4 %a, C2
    =>
  %r = icmp sgt i4 %x, -1

Also approximated in Alive2:
https://alive2.llvm.org/ce/z/u5hCcz
https://alive2.llvm.org/ce/z/__szVL

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

3 years agoRevert "[mlir][linalg] Support parsing attributes in named op spec"
Mehdi Amini [Mon, 11 Jan 2021 20:42:10 +0000 (20:42 +0000)]
Revert "[mlir][linalg] Support parsing attributes in named op spec"

This reverts commit df86f15f0c53c395dac5a14aba08745bc12b9b9b.

The gcc-5 build was broken by this change:

  mlir/tools/mlir-linalg-ods-gen/mlir-linalg-ods-gen.cpp:1275:77:   required from here
  /usr/include/c++/5/ext/new_allocator.h:120:4: error: no matching function for call to 'std::pair<const std::__cxx11::basic_string<char>, {anonymous}::TCParser::RegisteredAttr>::pair(llvm::StringRef&, {anonymous}::TCParser::RegisteredAttr'

3 years ago[mlir][CAPI] Introduce standard source layout for mlir-c dialect registration.
Stella Laurenzo [Sun, 10 Jan 2021 03:01:39 +0000 (19:01 -0800)]
[mlir][CAPI] Introduce standard source layout for mlir-c dialect registration.

* Registers a small set of sample dialects.
* NFC with respect to existing C-API symbols but some headers have been moved down a level to the Dialect/ sub-directory.
* Adds an additional entry point per dialect that is needed for dynamic discovery/loading.
* See discussion: https://llvm.discourse.group/t/dialects-and-the-c-api/2306/16

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

3 years agoEnable python bindings for tensor, shape and linalg dialects.
Stella Laurenzo [Sun, 10 Jan 2021 01:14:47 +0000 (17:14 -0800)]
Enable python bindings for tensor, shape and linalg dialects.

* We've got significant missing features in order to use most of these effectively (i.e. custom builders, region-based builders).
* We presently also lack a mechanism for actually registering these dialects but they can be use with contexts that allow unregistered dialects for further prototyping.

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

3 years ago[mlir][vector] Add side-effect information to different load/store ops
Thomas Raoux [Mon, 11 Jan 2021 18:35:30 +0000 (10:35 -0800)]
[mlir][vector] Add side-effect information to different load/store ops

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

3 years ago[NFC] Disallow unused prefixes under llvm/test/CodeGen
Mircea Trofin [Mon, 11 Jan 2021 17:34:45 +0000 (09:34 -0800)]
[NFC] Disallow unused prefixes under llvm/test/CodeGen

This patch finishes addressing unused prefixes under CodeGen: 2
remaining tests fixed, and then undo-ing the lit.local.cfg changes under
various subdirs and moving the policy under CodeGen.

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

3 years ago[tools] Mark output of tools as text if it is really text
Abhina Sreeskantharajan [Mon, 11 Jan 2021 20:13:40 +0000 (15:13 -0500)]
[tools] Mark output of tools as text if it is really text

This is a continuation of https://reviews.llvm.org/D67696. The following tools also need to set the OF_Text flag correctly.

  -   llvm-profdata
  -   llvm-link

Reviewed By: rnk

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

3 years ago[ADT] Add makeIntrusiveRefCnt helper function
Nathan James [Mon, 11 Jan 2021 20:12:53 +0000 (20:12 +0000)]
[ADT] Add makeIntrusiveRefCnt helper function

Works like std::make_unique but for IntrusiveRefCntPtr objects.
See https://lists.llvm.org/pipermail/llvm-dev/2021-January/147729.html

Reviewed By: dblaikie, MaskRay

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

3 years ago[mlir][IR][NFC] Move the definitions of Complex/Function/Integer/Opaque/TupleType...
River Riddle [Mon, 11 Jan 2021 19:55:09 +0000 (11:55 -0800)]
[mlir][IR][NFC] Move the definitions of Complex/Function/Integer/Opaque/TupleType to ODS

The type tablegen backend now has enough support to represent these types well enough, so we can now move them to be declaratively defined.

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

3 years ago[mlir][TypeDefGen] Add support for adding builders when generating a TypeDef
River Riddle [Mon, 11 Jan 2021 19:55:00 +0000 (11:55 -0800)]
[mlir][TypeDefGen] Add support for adding builders when generating a TypeDef

This allows for specifying additional get/getChecked methods that should be generated on the type, and acts similarly to how OpBuilders work. TypeBuilders have two additional components though:
* InferredContextParam
  - Bit indicating that the context parameter of a get method is inferred from one of the builder parameters
* checkedBody
  - A code block representing the body of the equivalent getChecked method.

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

3 years ago[mlir][ODS] Add a C++ abstraction for OpBuilders
River Riddle [Mon, 11 Jan 2021 19:54:51 +0000 (11:54 -0800)]
[mlir][ODS] Add a C++ abstraction for OpBuilders

This removes the need for OpDefinitionsGen to use raw tablegen API, and will also
simplify adding builders to TypeDefs as well.

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

3 years ago[NFC][AMDGPU] Clarify memory model support for volatile
Tony Tye [Sat, 9 Jan 2021 10:20:42 +0000 (10:20 +0000)]
[NFC][AMDGPU] Clarify memory model support for volatile

Reorder the AMDGPUUage description of the memory model code sequences
for volatile so clear that it applies independent of the nontemporal
setting.

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

3 years ago[libc++] Turn off auto-formatting of generated files. NFC.
Marek Kurdej [Mon, 11 Jan 2021 19:48:15 +0000 (20:48 +0100)]
[libc++] Turn off auto-formatting of generated files. NFC.

This adds `// clang-format off` in the auto-generated file to avoid lint warnings.

Reviewed By: ldionne, #libc

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

3 years ago[RISCV] Add scalable vector fcmp ISel patterns
Fraser Cormack [Wed, 6 Jan 2021 11:25:25 +0000 (11:25 +0000)]
[RISCV] Add scalable vector fcmp ISel patterns

Original patch by @rogfer01.

All ordered comparisons except ONE are supported natively, and all
unordered comparisons except UNE are expanded into sequences involving
explicit NaN checks and mask arithmetic.

Additionally, we expand GT,OGT,GE,OGE to their swapped-operand versions, and
pattern-match those back to the "original", swapping operands once more. This
way we catch both operations and both "vf" and "fv" forms with fewer patterns.

Also add support for floating-point splat_vector, with an optimization for
splatting fpimm0.

Authored-by: Roger Ferrer Ibanez <rofirrim@gmail.com>
Co-Authored-by: Fraser Cormack <fraser@codeplay.com>
Reviewed By: craig.topper

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

3 years ago[mlir] Add structural conversion to async dialect lowering.
Christian Sigg [Mon, 11 Jan 2021 19:34:44 +0000 (20:34 +0100)]
[mlir] Add structural conversion to async dialect lowering.

Lowering of async dialect uses a fixed type converter and therefore does not support lowering non-standard types.

This revision adds a structural conversion so that non-standard types in `!async.value`s can be lowered to LLVM before lowering the async dialect itself.

Reviewed By: ezhulenev

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

3 years ago[SystemZ][z/OS] Fix Permission denied pattern matching
Abhina Sreeskantharajan [Mon, 11 Jan 2021 19:27:10 +0000 (14:27 -0500)]
[SystemZ][z/OS] Fix Permission denied pattern matching

On z/OS, the error message "EDC5111I Permission denied." is not matched correctly in lit tests. This patch updates the check expression to match successfully.

Reviewed By: fanbo-meng

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

3 years agoFix minor build issue (NFC)
David Stuttard [Mon, 11 Jan 2021 19:20:47 +0000 (11:20 -0800)]
Fix minor build issue (NFC)

Change [x86] Fix tile register spill issue was causing problems for our build
using gcc-5.4.1

The problem was caused by this line:

for (const MachineInstr &MI : make_range(MIS.begin(), MI))

where MI was previously defined as a MachineBasicBlock iterator.

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

3 years agoIntroduce new quiet mode and new option handling for -print-changed.
Jamie Schmeiser [Mon, 11 Jan 2021 19:11:00 +0000 (14:11 -0500)]
Introduce new quiet mode and new option handling for -print-changed.
Summary:
Introduce a new mode of operation for -print-changed that only reports
after a pass changes the IR with all of the other messages suppressed (ie,
no initial IR and no messages about ignored, filtered or non-modifying
passes).

The option processing for -print-changed is changed to take an optional
string indicating options for print-changed. Initially, the only option
supported is quiet (as described above). This new quiet mode is specified
with -print-changed=quiet while -print-changed will continue to function
in the same way. It is intended that there will be more options in the
future.

Author: Jamie Schmeiser <schmeise@ca.ibm.com>
Reviewed By: aeubanks (Arthur Eubanks)
Differential Revision: https://reviews.llvm.org/D92589

3 years ago-funique-internal-linkage-names appends a hex md5hash suffix to the symbol name which...
Sriraman Tallam [Mon, 11 Jan 2021 19:08:36 +0000 (11:08 -0800)]
-funique-internal-linkage-names appends a hex md5hash suffix to the symbol name which is not demangler friendly, convert it to decimal.

Please see D93747 for more context which tries to make linkage names of internal
linkage functions to be the uniqueified names. This causes a problem with gdb
because breaking using the demangled function name will not work if the new
uniqueified name cannot be demangled. The problem is the generated suffix which
is a mix of integers and letters which do not demangle. The demangler accepts
either all numbers or all letters. This patch simply converts the hash to decimal.

There is no loss of uniqueness by doing this as the precision is maintained.
The symbol names get longer by a few characters though.

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

3 years ago[flang][openxx][NFC] Remove duplicated function to check required clauses
Valentin Clement [Mon, 11 Jan 2021 19:08:35 +0000 (14:08 -0500)]
[flang][openxx][NFC] Remove duplicated function to check required clauses

Remove duplicated function to check for required clauses on a directive. This was
still there from the merging of OpenACC and OpenMP common semantic checks and it can now be
removed so we use only one function.

Reviewed By: sameeranjoshi

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

3 years ago[mlir] Expose MemRef layout in Python bindings
Alex Zinenko [Fri, 8 Jan 2021 14:08:44 +0000 (15:08 +0100)]
[mlir] Expose MemRef layout in Python bindings

This wasn't possible before because there was no support for affine expressions
as maps. Now that this support is available, provide the mechanism for
constructing maps with a layout and inspecting it.

Rework the `get` method on MemRefType in Python to avoid needing an explicit
memory space or layout map. Remove the `get_num_maps`, it is too low-level,
using the length of the now-avaiable pseudo-list of layout maps is more
pythonic.

Depends On D94297

Reviewed By: mehdi_amini

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

3 years ago[mlir] More Python bindings for AffineMap
Alex Zinenko [Fri, 8 Jan 2021 12:36:27 +0000 (13:36 +0100)]
[mlir] More Python bindings for AffineMap

Now that the bindings for AffineExpr have been added, add more bindings for
constructing and inspecting AffineMap that consists of AffineExprs.

Depends On D94225

Reviewed By: mehdi_amini

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

3 years ago[mlir] Add Python bindings for AffineExpr
Alex Zinenko [Thu, 7 Jan 2021 10:09:09 +0000 (11:09 +0100)]
[mlir] Add Python bindings for AffineExpr

This adds the Python bindings for AffineExpr and a couple of utility functions
to the C API. AffineExpr is a top-level context-owned object and is modeled
similarly to attributes and types. It is required, e.g., to build layout maps
of the built-in memref type.

Reviewed By: mehdi_amini

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

3 years ago[AMDGPU] Deduplicate VOP tablegen asm & ins
Joe Nash [Tue, 5 Jan 2021 00:47:55 +0000 (19:47 -0500)]
[AMDGPU] Deduplicate VOP tablegen asm & ins

VOP3 and VOP DPP subroutines to generate input
operands and asm strings were essentially copy
pasted several times. They are deduplicated to
reduce the maintenance burden and allow faster
development.

Reviewed By: dp

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

Change-Id: I76225eed3c33239d9573351e0c8a0abfad0146ea

3 years ago[Hexagon] Custom-widen SETCC's operands
Krzysztof Parzyszek [Thu, 7 Jan 2021 15:30:56 +0000 (09:30 -0600)]
[Hexagon] Custom-widen SETCC's operands

The result cannot be widened, unfortunately, because widening vNi1
would depend on the context in which it appears (i.e. the type alone
is not sufficient to tell if it needs to be widened).

3 years ago[attributes] Add a facility for enforcing a Trusted Computing Base.
Sean Dooher [Mon, 11 Jan 2021 14:57:08 +0000 (06:57 -0800)]
[attributes]  Add a facility for enforcing a Trusted Computing Base.

Introduce a function attribute 'enforce_tcb' that prevents the function
from calling other functions without the same attribute. This allows
isolating code that's considered to be somehow privileged so that it could not
use its privileges to exhibit arbitrary behavior.

Introduce an on-by-default warning '-Wtcb-enforcement' that warns
about violations of the above rule.

Introduce a function attribute 'enforce_tcb_leaf' that suppresses
the new warning within the function it is attached to. Such leaf functions
may implement common functionality between the trusted and the untrusted code
but they require extra careful audit with respect to their capabilities.

Fixes after a revert in 419ef38a50293c58078f830517f5e305068dbee6:
Fix a test.
Add workaround for GCC bug (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67274).
Attribute the patch appropriately!

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

3 years ago[X86] Regenerate vector-constrained-fp-intrinsics.ll tests
Simon Pilgrim [Mon, 11 Jan 2021 18:12:25 +0000 (18:12 +0000)]
[X86] Regenerate vector-constrained-fp-intrinsics.ll tests

Adding missing libcall PLT qualifier

3 years ago[FastISel] NFC: Clean up unnecessary bookkeeping
Paul Robinson [Mon, 11 Jan 2021 17:33:55 +0000 (09:33 -0800)]
[FastISel] NFC: Clean up unnecessary bookkeeping

Now that we flush the local value map for every instruction, we don't
need any extra flushes for specific cases.  Also, LastFlushPoint is
not used for anything.  Follow-ups to #c161665 (D91734).

This reapplies #3fd39d3.

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

3 years ago[SystemZ] Minor NFC fix in SchedModels.
Jonas Paulsson [Mon, 11 Jan 2021 16:21:44 +0000 (10:21 -0600)]
[SystemZ]  Minor NFC fix in SchedModels.

The unused LRMux opcode was removed by 8f8c381, but a regexp still matched
for it in the scheduler files which is now removed.

Review: Ulrich Weigand

3 years ago[ELF] --exclude-libs: localize defined libcall symbols referenced by lto.tmp
Fangrui Song [Mon, 11 Jan 2021 17:33:21 +0000 (09:33 -0800)]
[ELF] --exclude-libs: localize defined libcall symbols referenced by lto.tmp

Fixes PR48681: after LTO, lto.tmp may reference a libcall symbol not in an IR
symbol table of any bitcode file. If such a symbol is defined in an archive
matched by a --exclude-libs, we don't correctly localize the symbol.

Add another `excludeLibs` after `compileBitcodeFiles` to localize such libcall
symbols. Unfortunately we have keep the existing one for D43126.

Using VER_NDX_LOCAL is an implementation detail of `--exclude-libs`, it does not
necessarily tie to the "localize" behavior.  `local:` patterns in a version
script can be omitted.
The `symbol ... has undefined version ...` error should not be exempted.
Ideally we should error as GNU ld does. https://issuetracker.google.com/issues/73020933

Reviewed By: psmith

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

3 years ago[FastISel] NFC: Remove obsolete -fast-isel-sink-local-values option
Paul Robinson [Mon, 11 Jan 2021 17:20:45 +0000 (09:20 -0800)]
[FastISel] NFC: Remove obsolete -fast-isel-sink-local-values option

This option is not used for anything after #c161665 (D91737).
This commit reapplies #a474657.

3 years ago[mlir][Linalg] Fix reshape fusion to reshape the outs instead of creating new tensors.
MaheshRavishankar [Mon, 11 Jan 2021 17:24:07 +0000 (09:24 -0800)]
[mlir][Linalg] Fix reshape fusion to reshape the outs instead of creating new tensors.

When fusing tensor_reshape ops with generic/indexed_Generic op, new
linalg.init_tensor operations were created for the `outs` of the fused
op. While correct (technically) it is better to just reshape the
original `outs` operands and rely on canonicalization of init_tensor
-> tensor_reshape to achieve the same effect.

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

3 years ago[mlir][vector] Add memory effects to transfer_read transfer_write ops
Thomas Raoux [Fri, 8 Jan 2021 17:46:41 +0000 (09:46 -0800)]
[mlir][vector] Add memory effects to transfer_read transfer_write ops

This allow more accurate modeling of the side effects and allow dead code
elimination to remove dead transfer ops.

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

3 years ago[NFC] Disallow unused prefixes in CodeGen/PowerPC tests.
Mircea Trofin [Mon, 11 Jan 2021 04:15:05 +0000 (20:15 -0800)]
[NFC] Disallow unused prefixes in CodeGen/PowerPC tests.

Also removed where applicable.

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

3 years ago[Clang][Docs] Fix ambiguity in clang-offload-bundler docs
Scott Linder [Fri, 8 Jan 2021 21:55:57 +0000 (21:55 +0000)]
[Clang][Docs] Fix ambiguity in clang-offload-bundler docs

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

3 years ago[mlir][Linalg] Fold init_tensor -> linalg.tensor_reshape.
MaheshRavishankar [Mon, 11 Jan 2021 17:21:39 +0000 (09:21 -0800)]
[mlir][Linalg] Fold init_tensor -> linalg.tensor_reshape.

Reshaping an init_tensor can be folded to a init_tensor op of the
final type.

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

3 years ago[X86][AVX] Attempt to fold vpermf128(op(x,i),op(y,i)) -> op(vpermf128(x,y),i)
Simon Pilgrim [Mon, 11 Jan 2021 16:59:07 +0000 (16:59 +0000)]
[X86][AVX] Attempt to fold vpermf128(op(x,i),op(y,i)) -> op(vpermf128(x,y),i)

If vpermf128/vpermi128 is acting on 2 similar 'inlane' ops, then try to perform the vpermf128 first which will allow us to merge the ops.

This will help us fix one of the regressions in D56387

3 years ago[FastISel] Flush local value map on every instruction
Paul Robinson [Mon, 11 Jan 2021 16:32:36 +0000 (08:32 -0800)]
[FastISel] Flush local value map on every instruction

Local values are constants or addresses that can't be folded into
the instruction that uses them. FastISel materializes these in a
"local value" area that always dominates the current insertion
point, to try to avoid materializing these values more than once
(per block).

https://reviews.llvm.org/D43093 added code to sink these local
value instructions to their first use, which has two beneficial
effects. One, it is likely to avoid some unnecessary spills and
reloads; two, it allows us to attach the debug location of the
user to the local value instruction. The latter effect can
improve the debugging experience for debuggers with a "set next
statement" feature, such as the Visual Studio debugger and PS4
debugger, because instructions to set up constants for a given
statement will be associated with the appropriate source line.

There are also some constants (primarily addresses) that could be
produced by no-op casts or GEP instructions; the main difference
from "local value" instructions is that these are values from
separate IR instructions, and therefore could have multiple users
across multiple basic blocks. D43093 avoided sinking these, even
though they were emitted to the same "local value" area as the
other instructions. The patch comment for D43093 states:

  Local values may also be used by no-op casts, which adds the
  register to the RegFixups table. Without reversing the RegFixups
  map direction, we don't have enough information to sink these
  instructions.

This patch undoes most of D43093, and instead flushes the local
value map after(*) every IR instruction, using that instruction's
debug location. This avoids sometimes incorrect locations used
previously, and emits instructions in a more natural order.

In addition, constants materialized due to PHI instructions are
not assigned a debug location immediately; instead, when the
local value map is flushed, if the first local value instruction
has no debug location, it is given the same location as the
first non-local-value-map instruction.  This prevents PHIs
from introducing unattributed instructions, which would either
be implicitly attributed to the location for the preceding IR
instruction, or given line 0 if they are at the beginning of
a machine basic block.  Neither of those consequences is good
for debugging.

This does mean materialized values are not re-used across IR
instruction boundaries; however, only about 5% of those values
were reused in an experimental self-build of clang.

(*) Actually, just prior to the next instruction. It seems like
it would be cleaner the other way, but I was having trouble
getting that to work.

This reapplies commits cf1c774d and dc35368c, and adds the
modification to PHI handling, which should avoid problems
with debugging under gdb.

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

3 years agoNFC: Use -LABEL more
Paul Robinson [Mon, 11 Jan 2021 15:03:08 +0000 (07:03 -0800)]
NFC: Use -LABEL more

There were a number of tests needing updates for D91734, and I added a
bunch of LABEL directives to help track down where those had to go.
These directives are an improvement independent of the functional
patch, so I'm committing them as their own separate patch.

3 years ago[clangd] Remove ScratchFS from tests
Nathan James [Mon, 11 Jan 2021 16:14:26 +0000 (16:14 +0000)]
[clangd] Remove ScratchFS from tests

This can lead to issues if files in the tmp directory we don't care about / control are found.
This was partially addressed in D94321, but this is a more permanent fix.
Fixes https://github.com/clangd/clangd/issues/354

Reviewed By: adamcz, sammccall

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

3 years ago[OpenMPOpt][WIP] Expand parallel region merging
Giorgis Georgakoudis [Mon, 11 Jan 2021 16:03:08 +0000 (08:03 -0800)]
[OpenMPOpt][WIP] Expand parallel region merging

The existing implementation of parallel region merging applies only to
consecutive parallel regions that have speculatable sequential
instructions in-between. This patch lifts this limitation to expand
merging with any sequential instructions in-between, except calls to
unmergable OpenMP runtime functions. In-between sequential instructions
in the merged region are sequentialized in a "master" region and any
output values are broadcasted to the following parallel regions and the
sequential region continuation of the merged region.

Reviewed By: jdoerfert

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

3 years ago[clangd] Fix -check mode doesn't respect any tidy configs.
Haojian Wu [Mon, 11 Jan 2021 14:48:27 +0000 (15:48 +0100)]
[clangd] Fix -check mode doesn't respect any tidy configs.

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

3 years ago[ARM] Update existing test case with +pauth targets
Ranjeet Singh [Mon, 11 Jan 2021 15:37:51 +0000 (15:37 +0000)]
[ARM] Update existing test case with +pauth targets

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

3 years ago[X86] Extend lzcnt-cmp tests to test on non-lzcnt targets
Simon Pilgrim [Mon, 11 Jan 2021 15:27:08 +0000 (15:27 +0000)]
[X86] Extend lzcnt-cmp tests to test on non-lzcnt targets

3 years ago[X86] Add nounwind to lzcnt-cmp tests
Simon Pilgrim [Mon, 11 Jan 2021 15:02:43 +0000 (15:02 +0000)]
[X86] Add nounwind to lzcnt-cmp tests

Remove unnecessary cfi markup

3 years ago[mlir] Fix gpu-to-llvm lowering for gpu.alloc with dynamic sizes.
Christian Sigg [Mon, 11 Jan 2021 13:46:26 +0000 (14:46 +0100)]
[mlir] Fix gpu-to-llvm lowering for gpu.alloc with dynamic sizes.

Reviewed By: ftynse

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

3 years agoRevert "[attributes] Add a facility for enforcing a Trusted Computing Base."
Nico Weber [Mon, 11 Jan 2021 14:51:06 +0000 (09:51 -0500)]
Revert "[attributes]  Add a facility for enforcing a Trusted Computing Base."

This reverts commit c163aae45ef6b7f3bd99601195d3ce4aad5850c6.
Doesn't compile on some bots
(http://lab.llvm.org:8011/#/builders/98/builds/3387/steps/9/logs/stdio),
breaks tests on bots where it does compile
(http://45.33.8.238/linux/36843/step_7.txt).

3 years ago[VPlan] Unify value/recipe printing after VPDef transition.
Florian Hahn [Mon, 23 Nov 2020 15:44:50 +0000 (15:44 +0000)]
[VPlan] Unify value/recipe printing after VPDef transition.

This patch unifies the way recipes and VPValues are printed after the
transition to VPDef.

VPSlotTracker has been updated to iterate over all recipes and all
their defined values to number those. There is no need to number
values in Value2VPValue.

It also updates a few places that only used slot numbers for
VPInstruction. All recipes now can produce numbered VPValues.

3 years ago[attributes] Add a facility for enforcing a Trusted Computing Base.
Artem Dergachev [Mon, 11 Jan 2021 12:52:04 +0000 (04:52 -0800)]
[attributes]  Add a facility for enforcing a Trusted Computing Base.

Introduce a function attribute 'enforce_tcb' that prevents the function
from calling other functions without the same attribute. This allows
isolating code that's considered to be somehow privileged so that it could not
use its privileges to exhibit arbitrary behavior.

Introduce an on-by-default warning '-Wtcb-enforcement' that warns
about violations of the above rule.

Introduce a function attribute 'enforce_tcb_leaf' that suppresses
the new warning within the function it is attached to. Such leaf functions
may implement common functionality between the trusted and the untrusted code
but they require extra careful audit with respect to their capabilities.

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

3 years ago[DAGCombiner] Use getVectorElementCount inside visitINSERT_SUBVECTOR
Joe Ellis [Tue, 15 Dec 2020 17:20:11 +0000 (17:20 +0000)]
[DAGCombiner] Use getVectorElementCount inside visitINSERT_SUBVECTOR

This avoids TypeSize-/ElementCount-related warnings.

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

3 years ago[mlir][linalg] Support permutation when lowering to loop nests
Lei Zhang [Mon, 11 Jan 2021 14:08:21 +0000 (09:08 -0500)]
[mlir][linalg] Support permutation when lowering to loop nests

Linalg ops are perfect loop nests. When materializing the concrete
loop nest, the default order specified by the Linalg op's iterators
may not be the best for further CodeGen: targets frequently need
to plan the loop order in order to gain better data access. And
different targets can have different preferences. So there should
exist a way to control the order.

Reviewed By: nicolasvasilache

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

3 years ago[mlir][linalg] Support parsing attributes in named op spec
Lei Zhang [Mon, 11 Jan 2021 13:50:00 +0000 (08:50 -0500)]
[mlir][linalg] Support parsing attributes in named op spec

With this, now we can specify a list of attributes on named ops
generated from the spec. The format is defined as

```
attr-id ::= bare-id (`?`)?
attr-typedef ::= type (`[` `]`)?
attr-def ::= attr-id `:` attr-typedef

tc-attr-def ::= `attr` `(` attr-def-list `)`
tc-def ::= `def` bare-id
  `(`tensor-def-list`)` `->` `(` tensor-def-list`)`
  (tc-attr-def)?
```

For example,

```
ods_def<SomeCppOp>
def some_op(...) -> (...)
attr(
  f32_attr: f32,
  i32_attr: i32,
  array_attr : f32[],
  optional_attr? : f32
)
```

where `?` means optional attribute and `[]` means array type.

Reviewed By: hanchung, nicolasvasilache

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

3 years ago[flang][driver] Copy input files into a temp dir when testing
Andrzej Warzynski [Thu, 7 Jan 2021 17:49:38 +0000 (17:49 +0000)]
[flang][driver] Copy input files into a temp dir when testing

The following frontend driver invocation will generate 2 output files
in the same directory as the input files:
```
flang-new -fc1 input-1.f input-2.f
```
This is the desired behaviour. However, when testing we need to make
sure that we don't pollute the source directory. To this end, copy test
input files into a temporary directory.

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

3 years ago[mlir] Make GpuAsyncRegion pass depend on async dialect.
Christian Sigg [Mon, 11 Jan 2021 12:56:35 +0000 (13:56 +0100)]
[mlir] Make GpuAsyncRegion pass depend on async dialect.

Do not cache gpu.async.token type so that the pass can be created before the GPU dialect is registered.

Reviewed By: ftynse

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

3 years ago[mlir] Remove unnecessary llvm.mlir.cast in AsyncToLLVM lowering.
Christian Sigg [Mon, 11 Jan 2021 12:25:23 +0000 (13:25 +0100)]
[mlir] Remove unnecessary llvm.mlir.cast in AsyncToLLVM lowering.

Reviewed By: ftynse

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

3 years ago[AMDGPU] Fix a urem combine test to test what it was supposed to
Jay Foad [Mon, 11 Jan 2021 13:28:34 +0000 (13:28 +0000)]
[AMDGPU] Fix a urem combine test to test what it was supposed to

3 years ago[ARM] Add uses for locals introduced for debug messages. NFC.
Stephan Herhut [Mon, 11 Jan 2021 11:32:25 +0000 (12:32 +0100)]
[ARM] Add uses for locals introduced for debug messages. NFC.

This adds uses for locals introduced for new debug messages for the load store optimizer. Those locals are only used on debug statements and otherwise create unused variable warnings.

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

3 years ago[X86][SSE] Add 'vectorized sum' test patterns
Simon Pilgrim [Mon, 11 Jan 2021 12:51:03 +0000 (12:51 +0000)]
[X86][SSE] Add 'vectorized sum' test patterns

These are often generated when building a vector from the reduction sums of independent vectors.

I've implemented some typical patterns from various v4f32/v4i32 based off current codegen emitted from the vectorizers, although these tests are more about tweaking some hadd style backend folds to handle whatever the vectorizers/vectorcombine throws at us...

3 years ago[lldb] Disable PipeTest.OpenAsReader on windows
Pavel Labath [Mon, 11 Jan 2021 12:15:01 +0000 (13:15 +0100)]
[lldb] Disable PipeTest.OpenAsReader on windows

This test seems to be broken there (which is not totally surprising as
this functionality was never used on windows). Disable the test while I
investigate.

3 years ago[obj2yaml][test] - Improve and fix section-group.yaml test.
Georgii Rymar [Wed, 23 Dec 2020 10:40:58 +0000 (13:40 +0300)]
[obj2yaml][test] - Improve and fix section-group.yaml test.

It has multiple issues fixed by this patch:
1) It shouldn't test how llvm-readelf/yaml2obj works.
2) It should use "-NEXT" prefix for check lines.
3) It can use YAML macros, that allows to use a single YAML.
4) It should probably test the case when a group member is a null section.

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

3 years ago[VPlan] Move initial quote emission from ::print to ::dumpBasicBlock.
Florian Hahn [Mon, 11 Jan 2021 12:20:04 +0000 (12:20 +0000)]
[VPlan] Move initial quote emission from ::print to ::dumpBasicBlock.

This means there will be no stray " when printing individual recipes
using print()/dump() in a debugger, for example.

3 years ago[llvm-readelf/obj] - Index phdrs and relocations from 0 when reporting warnings.
Georgii Rymar [Thu, 24 Dec 2020 13:20:07 +0000 (16:20 +0300)]
[llvm-readelf/obj] - Index phdrs and relocations from 0 when reporting warnings.

As was mentioned in comments here:
https://reviews.llvm.org/D92636#inline-864967

we are not consistent and sometimes index things from 0, but sometimes
from 1 in warnings.

This patch fixes 2 places: messages reported for
program headers and messages reported for relocations.

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

3 years ago[clang][AArch64][SVE] Avoid going through memory for coerced VLST return values
Joe Ellis [Fri, 8 Jan 2021 11:44:15 +0000 (11:44 +0000)]
[clang][AArch64][SVE] Avoid going through memory for coerced VLST return values

VLST return values are coerced to VLATs in the function epilog for
consistency with the VLAT ABI. Previously, this coercion was done
through memory. It is preferable to use the
llvm.experimental.vector.insert intrinsic to avoid going through memory
here.

Reviewed By: c-rhodes

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

3 years ago[obj2yaml] - Fix the crash in getUniquedSectionName().
Georgii Rymar [Wed, 23 Dec 2020 10:29:01 +0000 (13:29 +0300)]
[obj2yaml] - Fix the crash in getUniquedSectionName().

`getUniquedSectionName(const Elf_Shdr *Sec)` assumes that
`Sec` is not `nullptr`.

I've found one place in `getUniquedSymbolName` where it is
not true (because of that we crash when trying to dump
unnamed null section symbols).

Patch fixes the crash and changes the signature of the
`getUniquedSectionName` section to accept a reference.

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

3 years ago[VE] Support additional VMRGW and VMV intrinsic instructions
Kazushi (Jam) Marukawa [Fri, 8 Jan 2021 11:41:37 +0000 (20:41 +0900)]
[VE] Support additional VMRGW and VMV intrinsic instructions

Support missing VMRGW and VMV intrinsic instructions and add regression
tests.

Reviewed By: simoll

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

3 years ago[VE] Support intrinsic to isnert/extract_subreg of v512i1
Kazushi (Jam) Marukawa [Fri, 8 Jan 2021 11:29:42 +0000 (20:29 +0900)]
[VE] Support intrinsic to isnert/extract_subreg of v512i1

Support insert/extract_subreg intrinsic instructions for v512i1
registers and add regression tests.

Reviewed By: simoll

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

3 years ago[X86][SSE] Add missing SSE test coverage for permute(hop,hop) folds
Simon Pilgrim [Mon, 11 Jan 2021 11:24:20 +0000 (11:24 +0000)]
[X86][SSE] Add missing SSE test coverage for permute(hop,hop) folds

Should help avoid bugs like reported in rG80dee7965dff