diff options
author | Roman Lebedev <lebedev.ri@gmail.com> | 2021-08-27 20:15:45 +0300 |
---|---|---|
committer | Roman Lebedev <lebedev.ri@gmail.com> | 2021-08-27 20:26:53 +0300 |
commit | 6734018041e408fe3cad13f2782d8048437de405 (patch) | |
tree | e9643b042947d5499f4f8dee9e50ed7bf8a097f5 /llvm/lib/Transforms/Utils/LoopUtils.cpp | |
parent | 54e8cae565294059048f8dd88958c4a2a317a1c2 (diff) | |
download | llvm-6734018041e408fe3cad13f2782d8048437de405.zip llvm-6734018041e408fe3cad13f2782d8048437de405.tar.gz llvm-6734018041e408fe3cad13f2782d8048437de405.tar.bz2 |
[Codegen][X86] EltsFromConsecutiveLoads(): if only have AVX1, ensure that the "load" is actually foldable (PR51615)
This fixes another reproducer from https://bugs.llvm.org/show_bug.cgi?id=51615
And again, the fix lies not in the code added in D105390
In this case, we completely don't check that the "broadcast-from-mem" we create
can actually fold the load. In this case, it's operand was not a load at all:
```
Combining: t16: v8i32 = vector_shuffle<0,u,u,u,0,u,u,u> t14, undef:v8i32
Creating new node: t29: i32 = undef
RepeatLoad:
t8: i32 = truncate t7
t7: i64 = extract_vector_elt t5, Constant:i64<0>
t5: v2i64,ch = load<(load (s128) from %ir.arg)> t0, t2, undef:i64
t2: i64,ch = CopyFromReg t0, Register:i64 %0
t1: i64 = Register %0
t4: i64 = undef
t3: i64 = Constant<0>
Combining: t15: v8i32 = undef
```
Reviewed By: RKSimon
Differential Revision: https://reviews.llvm.org/D108821
Diffstat (limited to 'llvm/lib/Transforms/Utils/LoopUtils.cpp')
0 files changed, 0 insertions, 0 deletions