- Apr 15, 2020
-
-
Benjamin Kramer authored
No functionality change intended.
-
David Truby authored
This removes the current undefined behavior where the function has a return type but no return statement. Fixes: https://bugs.llvm.org/show_bug.cgi?id=45530
-
Simon Moll authored
-
Simon Moll authored
-
Denis Antrushin authored
To simplify future work on statepoint representation, hide direct access to statepoint field indices and provide getters for them. Add getters for couple more statepoint fields. This also fixes two bugs in MachineVerifier for statepoint: First, the `break` statement was falling out of `if` statement scope, thus disabling following checks. Second, it was incorrectly accessing some fields like CallingConv - StatepointOpers gives index to their value directly, not to preceeding field type encoding. Reviewed By: skatkov Differential Revision: https://reviews.llvm.org/D78119
-
Kirill Bobyrev authored
Summary: Before: USAGE: dexp [options] --index-path Path to the index After: USAGE: dexp [options] <INDEX FILE> Reviewers: sammccall Reviewed By: sammccall Subscribers: jkorous, arphaman, kadircet, usaxena95, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D78089
-
Florian Hahn authored
-
Jonas Paulsson authored
adjustSubwordCmp() should not optimize a load of an i1 value. This is achieved by checking that the size and store-size of the MemoryVT are the same. Fixes https://bugs.llvm.org/show_bug.cgi?id=45511. Review: Ulrich Weigand Differential Revision: https://reviews.llvm.org/D78187
-
Simon Pilgrim authored
Add SSE + AVX512 targets Add some illegal type store tests
-
Benjamin Kramer authored
No functionality change intended.
-
Florian Hahn authored
After introducing VPWidenSelectRecipe, the duplicated logic can be shared. Reviewers: gilr, rengolin, Ayal, hsaito Reviewed By: Ayal Differential Revision: https://reviews.llvm.org/D77973
-
Pavel Labath authored
-
Pavel Labath authored
Summary: If we have an (invalid) relocation which relocates bytes which partially lie outside the range of the relocated section, the getRelocatedValue would return confusing results. It would first read zero (because that's what the underlying DataExtractor api does for out-of-bounds reads), and then relocate that zero anyway. A more appropriate behavior is to return zero straight away. This is what this patch does. Reviewers: dblaikie, jhenderson Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D78113
-
Florian Hahn authored
We can eliminate MemoryDefs of objects not accessible after the function returns (e.g. alloca), if there are no reads between the MemoryDef and any function exits. We can stop traversing paths that completely overwrite the memory location of the MemoryDef. This patch was split off D73763. Reviewers: dmgreen, bryant, asbirlea, Tyker, efriedma, george.burgess.iv Reviewed By: asbirlea, george.burgess.iv Differential Revision: https://reviews.llvm.org/D77736
-
Sam Parker authored
Make the non-exchanging versions of the multiply add/sub instructions validForTailPredication. Differential Revision: https://reviews.llvm.org/D77648
-
Sameer Sahasrabuddhe authored
This is related to commit 8c11bc0c which introduces the FixIrreducible pass. The warning seems hard to reproduce locally. The latest attempt ought to work.
-
Nandor Licker authored
Summary: Updated the documentation to better reflect features implemented on the constexpr branch at https://github.com/nandor/llvm-project and extended the TODO list with known missing features Reviewers: rsmith, Bigcheese, dexonsmith, jfb Subscribers: lebedev.ri, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D75726
-
Georgii Rymar authored
We have a few unused/broken FileCheck prefixes in `llvm/test/tools`. This patch fixes it. Differential revision: https://reviews.llvm.org/D78110
-
Simon Moll authored
Differential Revision: https://reviews.llvm.org/D78188
-
LLVM GN Syncbot authored
-
Sameer Sahasrabuddhe authored
An irreducible SCC is one which has multiple "header" blocks, i.e., blocks with control-flow edges incident from outside the SCC. This pass converts an irreducible SCC into a natural loop by introducing a single new header block and redirecting all the edges on the original headers to this new block. This is a useful workaround for a limitation in the structurizer which, which produces incorrect control flow in the presence of irreducible regions. The AMDGPU backend provides an option to enable this pass before the structurizer, which may eventually be enabled by default. Reviewed By: nhaehnle Differential Revision: https://reviews.llvm.org/D77198 This restores commit 2ada8e25. Originally reverted with commit 44e09b59.
-
Florian Hahn authored
Handling LoadInst and StoreInst in tryToWiden seems a bit counter-intuitive, as there is only an assertion for them and in no case VPWidenRefipes are created for them. I think it makes sense to move the assertion to handleReplication, where the non-widened loads and store are handled. Reviewers: gilr, rengolin, Ayal, hsaito Reviewed By: Ayal Differential Revision: https://reviews.llvm.org/D77972
-
Alex Zinenko authored
Similarly to actual LLVM IR, and to `llvm.mlir.func`, allow the custom syntax of `llvm.mlir.global` to omit the linkage keyword. If omitted, the linkage is assumed to be external. This makes the modeling of globals in the LLVM dialect more consistent, both within the dialect and with LLVM IR. Differential Revision: https://reviews.llvm.org/D78096
-
Uday Bondhugula authored
Introduce mlir::applyOpPatternsAndFold which applies patterns as well as any folding only on a specified op (in contrast to applyPatternsAndFoldGreedily which applies patterns only on the regions of an op isolated from above). The caller is made aware of the op being folded away or erased. Depends on D77485. Differential Revision: https://reviews.llvm.org/D77487
-
Jan Kratochvil authored
As requested by @labath in https://reviews.llvm.org/D73206#1949516 providing DWARF index callbacks refactorization. Differential Revision: https://reviews.llvm.org/D77327
-
Shengchen Kan authored
As we disscussed in D77971, we haven't confirmed that if putting instructions in a non-executable section is an undefined behaviour. To make things easier to go on, we mark these sections executable in test file align-branch-section-size.s.
-
Kazushi (Jam) Marukawa authored
Summary: Changing all mnemonic to match assembly instructions to simplify mnemonic naming rules. This time update all fixed-point arithmetic instructions. This also corrects smax/smin code generations. Reviewed By: simoll Differential Revision: https://reviews.llvm.org/D77856
-
Raphael Isemann authored
Summary: The formatters code has a lot of 'reason' or 'why' values that we keep or-ing FormatterChoiceCriterion enum values into. These values are only read by a single log statement and don't have any functional purpose. It also seems the implementation is not finished (for example, display names and type names don't have any dedicated enum values). Also everything is of course not tested or documented. Let's just remove all of this. Reviewers: labath, JDevlieghere, jingham, davide, vsk Reviewed By: labath, vsk Subscribers: JDevlieghere Differential Revision: https://reviews.llvm.org/D77968
-
Gil Rapaport authored
Fix an assert introduced in 41ed5d85: a phi with a single predecessor and a mask is a valid case which is already supported by the code. Differential Revision: https://reviews.llvm.org/D78115
-
Kadir Cetinkaya authored
-
Kadir Cetinkaya authored
-
Kadir Cetinkaya authored
Summary: This reduces memory usage by dynamic index from more than 400MB to 32MB when all files in clang-tools-extra/clangd/*.cpp are active in clangd. Reviewers: sammccall Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D77732
-
Richard Smith authored
In passing, also generalize the mechanism used to allow Decl's printName functions to override qualified name printing.
-
Richard Smith authored
attributes to support any kind of declaration. In preparation for adding a third kind.
-
LLVM GN Syncbot authored
-
Sameer Sahasrabuddhe authored
This reverts commit 2ada8e25. Buildbots produced compilation errors which I was not able to quickly reproduce locally. Need more time to investigate.
-
Richard Smith authored
We generally prefer to have the ASTReader directly set fields rather than including public setter functions.
-
LLVM GN Syncbot authored
-
Sameer Sahasrabuddhe authored
An irreducible SCC is one which has multiple "header" blocks, i.e., blocks with control-flow edges incident from outside the SCC. This pass converts an irreducible SCC into a natural loop by introducing a single new header block and redirecting all the edges on the original headers to this new block. This is a useful workaround for a limitation in the structurizer which, which produces incorrect control flow in the presence of irreducible regions. The AMDGPU backend provides an option to enable this pass before the structurizer, which may eventually be enabled by default. Reviewed By: nhaehnle Differential Revision: https://reviews.llvm.org/D77198
-
Nathan Ridge authored
Summary: Fixes https://github.com/clangd/clangd/issues/297 Subscribers: ilya-biryukov, jkorous, arphaman, kadircet, usaxena95, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D76896
-