- Apr 19, 2020
-
-
Simon Pilgrim authored
-
Florian Hahn authored
This makes it easier to extend the merge options in the future and also reduces the risk of accidentally setting a wrong option. Reviewers: efriedma, nikic, reames, davide Reviewed By: efriedma Differential Revision: https://reviews.llvm.org/D78368
-
Uday Bondhugula authored
Update misleading line in conclusions. Although the application to IR objects is stated earlier, the concluding section contradicts it in isolation. Differential Revision: https://reviews.llvm.org/D78446
-
Uday Bondhugula authored
This change is in line with MLIR's coding style https://mlir.llvm.org/getting_started/DeveloperGuide/ and also consistent with the dominance methods in LLVM. Differential Revision: https://reviews.llvm.org/D78445
-
Craig Topper authored
-
Craig Topper authored
Noticed while looking for CallSite.h uses to remove.
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Carl Ritson authored
Summary: Add getBase accessor so that underlying tree can be manipulated in a similar manner to MachineDominatorTree. Reviewers: kuhar, arsenm, hliao, nhaehnle Reviewed By: kuhar Subscribers: lkail, mgorny, hiraditya, wdng, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D77967
-
Mircea Trofin authored
Incorporated feedback from https://reviews.llvm.org/D78414
-
LemonBoy authored
According to https://www.ibm.com/support/knowledgecenter/ssw_aix_72/assembler/idalangref_rldicl_rletdw_instrs.html rldicl should not be used when targeting 32bit CPUs. Reviewed By: #powerpc, nemanjai, MaskRay Differential Revision: https://reviews.llvm.org/D77946
-
Nico Weber authored
-
Lang Hames authored
-
Lang Hames authored
This can be used to reduce boilerplate code, especially when defining absolute symbols.
-
Lang Hames authored
LLJIT::defineAbsolute did not mangle its Name argument, which is inconsistent with the behavior of other LLJIT methods (e.g. lookup). Since it is currently unused anyway, this commit replaces it with a generic 'define' convenience method for adding MaterializationUnits to the main JITDylib. This simplifies use of the generic absoluteSymbols function (as well as the symbolAlias, reexports and other functions that generate MaterializationUnits) with LLJIT.
-
Ayal Zaks authored
First-order recurrences require special treatment when they are live-out; such treatment is provided by fixFirstOrderRecurrence(), so they should be included in AllowedExit set. (Should probably have been included originally in D16197.) Fixes PR45526: AllowedExit set is used by prepareToFoldTailByMasking() to check whether the treatment for live-outs also holds when folding the tail, which is not (yet) the case for first-order recurrences. Differential Revision: https://reviews.llvm.org/D78210
-
Craig Topper authored
With a tweak to avoid a linker error for passing MaxAlignmentExponent by reference to std::min.
-
Simon Pilgrim authored
-
Nikita Popov authored
ConstVal is not an owned pointer, so setting it to nullptr is not actually doing anything. If we switch to a state that does not use ConstVal, the value does not matter. Split out from D78425.
-
Nikita Popov authored
When running IPSCCP on a module with many small functions, memory usage is dominated by PredicateInfo, which is a huge structure (partially due to some unfortunate nested SmallVector use). However, most of it is actually only temporary state needed to build predicate info, and does not need to be retained after initial construction. This patch factors out the predicate building logic and state into a separate PrediceInfoBuilder, with the extra bonus that it does not need to live in the header anymore. Differential Revision: https://reviews.llvm.org/D78326
-
Craig Topper authored
This reverts commit e00cfe25. Seems to be causing a linker error on the build bots.
-
- Apr 17, 2020
-
-
kpdev authored
Error is raised because of using -Werror=convertion
-
- Apr 19, 2020
-
-
Craig Topper authored
We previously clamped the trailing zero count to 31 bits. And then clamped the final alignment to MaximumAlignment which is 1 << 29. This patch simplifies this to just clamp the trailing zero to 29 using MaxAlignmentExponent. I was looking into changing this function to use Align/MaybeAlign and noticed this. Differential Revision: https://reviews.llvm.org/D78418
-
LemonBoy authored
This allows the representation of arbitrarily large enumeration values. See https://lists.llvm.org/pipermail/llvm-dev/2017-December/119475.html for context. Reviewed By: andrewrk, aprantl, MaskRay Differential Revision: https://reviews.llvm.org/D62475
-
Uday Bondhugula authored
Rename mlir::tileCodeGen -> mlir::tilePerfectlyNested to be consistent. NFC clean up tiling utility code, drop dead code, better comments. Expose isPerfectlyNested and reuse. Differential Revision: https://reviews.llvm.org/D78423
-
Mircea Trofin authored
Summary: Depends on D78395. Reviewers: craig.topper, dblaikie, wmi, davidxl Subscribers: eraman, hiraditya, haicheng, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D78414
-
Florian Hahn authored
There are no lattice values associated with store instructions directly. They will never get marked as overdefined.
-
Uday Bondhugula authored
Make mlir::isPerfectlyNested more efficient; use O(1) check instead of O(N) size() method. Differential Revision: https://reviews.llvm.org/D78428
-
- Apr 18, 2020
-
-
Markus Böck authored
Currently C++ symbols are demangled in the symbol table as well as in the disassembly and relocations. This patch adds demangling of C++ symbols in targets of calls and branches making it easier to decipher control flow in disassembly. This also matches up with GNUobjdump's behavior Reviewed By: MaskRay Differential Revision: https://reviews.llvm.org/D77957
-
Louis Dionne authored
This was originally committed as f8452ddf and reverted in 7cb1aa9d. The issue was that shell builtins were being escaped too, and apparently Bash won't execute a builtin when it is quoted e.g. '!'. Instead, it thinks it's a command and it can't find it. Re-committing the change with that issue fixed.
-
Tobias Hieta authored
See http://lists.llvm.org/pipermail/llvm-dev/2020-April/140549.html For the record, GNU ld changed to 64k max page size in 2014 https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=7572ca8989ead4c3425a1500bc241eaaeffa2c89 "[RFC] ld/ARM: Increase maximum page size to 64kB" Android driver forced 4k page size in AArch64 (D55029) and ARM (D77746). A binary linked with max-page-size=4096 does not run on a system with a higher page size configured. There are some systems out there that do this and it leads to the binary getting `Killed!` by the kernel. In the non-linker-script cases, when linked with -z noseparate-code (default), the max-page-size increase should not cause any size difference. There may be some VMA usage differences, though. Reviewed By: psmith, MaskRay Differential Revision: https://reviews.llvm.org/D77330
-
Denis Khalikov authored
Summary: Workgroup size is written into the kernel. So to properly modelling vulkan launch, we have to skip local workgroup size for vulkan launch call op. Differential Revision: https://reviews.llvm.org/D78307
-
vgxbj authored
This change enables getSymbolFlags() to return errors which benefit error reporting in clients. Differential Revision: https://reviews.llvm.org/D77860
-
Florian Hahn authored
This patch adds a few additional test cases with cases subsequent patches will improve on.
-
Florian Hahn authored
There are no lattice values associated with return instructions directly. They will never get marked as overdefined.
-