- Oct 10, 2020
-
-
Jay Foad authored
Following on from D88890, this makes the newly added patterns conditional on NoFP32Denormals. mad/mac f32 instructions always flush denormals regardless of the MODE register setting, and I believe the legacy variants do the same. Differential Revision: https://reviews.llvm.org/D89123
-
Tres Popp authored
Without this PatternRewriting infrastructure does not know of modifications and cannot properly legalize nor rollback changes. Differential Revision: https://reviews.llvm.org/D89129
-
- Oct 09, 2020
-
-
Simon Pilgrim authored
[InstCombine] Support lshr(trunc(lshr(x,c1)), c2) -> trunc(lshr(lshr(x,c1),c2)) uniform vector tests FoldShiftByConstant is hardcoded for scalar/uniform outer shift amounts atm so that needs to be fixed first to support non-uniform cases
-
Simon Pilgrim authored
-
Eugene Zhulenev authored
Async execute operation can take async arguments as dependencies. Change `async.execute` custom parser/printer format to use `%value as %unwrapped: !async.value<!type>` sytax. Reviewed By: mehdi_amini, herhut Differential Revision: https://reviews.llvm.org/D88601
-
Andrzej Warzynski authored
Reverts one breaking change introduced in https://reviews.llvm.org/D88846. Differential Revision: https://reviews.llvm.org/D89111
-
David Green authored
There were some existing tests that were not super useful. New ones are added for testing MVE specific patterns.
-
Scott Linder authored
Reformat to avoid unrelated changes in diff of future patch. Committed as obvious.
-
Simon Pilgrim authored
Note: we already fold srem to undef if any denominator vector element is undef.
-
Krzysztof Parzyszek authored
-
Sanjay Patel authored
There might be a better way to specify the pre-conditions, but this is hopefully clearer than the way it was written: https://rise4fun.com/Alive/Jhk3 Pre: C2 < 0 && isShiftedMask(C2) && (C1 == C1 & C2) %a = and %x, C2 %r = add %a, C1 => %a2 = add %x, C1 %r = and %a2, C2
-
Simon Pilgrim authored
[InstCombine] Add tests for X shift (A srem B) -> X shift (A and B-1) pow2 nonuniform constant vectors
-
Anastasia Stulova authored
Extended -cl-std/std flag with CL3.0 and added predefined version macros. Patch by Anton Zabaznov (azabaznov)! Tags: #clang Differential Revision: https://reviews.llvm.org/D88300
-
Louis Dionne authored
To make sure we don't store a mutable object (which could be modified by outside code without us noticing) as the cache key, we pickle the cache key to get a byte stream. If two keys are unequal, we know for sure they will not have the same pickling. And if they are equal, there's a large chance they will have the same pickling. If they don't, we might end up not reusing a cached entry when we could have, but at least the behavior we'll have is semantically correct.
-
Irina Dobrescu authored
[mlir][openmp][NFC]Remove unnecessary brackets and rephrase ParallelOp description in mlir definition Differential Revision: https://reviews.llvm.org/D88740
-
Sam McCall authored
-
Fanbo Meng authored
z/OS defaults to 16 bytes for __attribute__((aligned)), modify the test to differentiate between z/OS and Linux on s390x. Reviewed By: abhina.sreeskantharajan Differential Revision: https://reviews.llvm.org/D89127
-
Louis Dionne authored
-
Tres Popp authored
Without this, legalization might not recursively handle child ops properly. Additionally, this is required for pattern rewriting to properly rollback conversions. Differential Revision: https://reviews.llvm.org/D89122
-
LLVM GN Syncbot authored
-
Stefan Pintilie authored
The Callbacks.cpp test was taking a long time to compile on some build bots causing timeouts. This patch splits up that test into five separate cpp files and a header file. Reviewed By: gribozavr2 Differential Revision: https://reviews.llvm.org/D88886
-
Florian Hahn authored
We cannot guarantee that the replacement expression is loop-invariant in all AddRecs in the source expression. Use a rewriter that skips AddRecExpr for now. Fixes PR47776.
-
Simon Pilgrim authored
-
Alexander Kornienko authored
Reviewed By: hokein Differential Revision: https://reviews.llvm.org/D89117
-
Simon Pilgrim authored
Allows us to drop the !isa<ConstantExpr> check.
-
Stephan Herhut authored
The updated version of kernel outlining did not handle cases correctly where an operand of a candidate for sinking itself was defined by an operation that is a sinking candidate. In such cases, it could happen that sunk operations were inserted in the wrong order, breaking ssa properties. Differential Revision: https://reviews.llvm.org/D89112
-
Simon Pilgrim authored
-
Martin Storsjö authored
This option just takes a single symbol name per invocation of the option. Differential Revision: https://reviews.llvm.org/D89007
-
Konstantin Zhuravlyov authored
Differential Revision: https://reviews.llvm.org/D89091
-
Ben Dunbobbin authored
Differential Revision: https://reviews.llvm.org/D86828
-
Ben Dunbobbin authored
Pushed by accident :( This reverts commit 4eb627ed.
-
Ben Dunbobbin authored
-
Sam McCall authored
(Required for CodeActionContext.only) Differential Revision: https://reviews.llvm.org/D88724
-
Roman Lebedev authored
I messed up runlines in the original commit.
-
Jonas Toth authored
This patch extracts the ExprMutAnalyzer changes from https://reviews.llvm.org/D54943 into its own revision for simpler review and more atomic changes. The analysis results are improved. Nested expressions (e.g. conditional operators) are now detected properly. Some edge cases, especially template induced imprecisions are improved upon. Reviewed By: aaron.ballman Differential Revision: https://reviews.llvm.org/D88088
-
Nithin Vadukkumchery Rajendrakumar authored
Summary: Enabling warning after dereferece of smart pointer after branching on unknown inner pointer. Reviewers: NoQ, Szelethus, vsavchenko, xazax.hun Reviewed By: NoQ Subscribers: martong, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D87043
-
Max Kazantsev authored
IV widening is sometimes a strictly harmful transform (some examples of this are shown in tests 11, 12 in widen-loop-comp.ll). One of the reasons of this is that sometimes SCEV fails to prove some facts after part of guards has been widened. Though each single such case looks like a bug that can be addressed, it seems that disabling of IV widening may be profitable in some cases. We want to have an option to do so. By default, existing behavior is preserved and IV widening is on.
-
James Molloy authored
When attempting to compute a differential orderIndex we were calculating the bailout condition correctly, but then an errant "+ 1" meant the orderIndex we created was invalid. Added test. Reviewed By: ftynse Differential Revision: https://reviews.llvm.org/D89115
-
Roman Lebedev authored
Buildbots aren't happy, need to investigate. This reverts commit 32cc8f79.
-
Jonas Paulsson authored
Since AGR clobbers CC it should not be used here. Fixes https://bugs.llvm.org/show_bug.cgi?id=47736. Review: Ulrich Weigand Differential Revision: https://reviews.llvm.org/D89034
-