diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2014-08-03 22:48:23 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2014-08-03 22:48:23 +0000 |
| commit | 2ef67440fc1e038c52a6bb6d874f34186e6b3b31 (patch) | |
| tree | 6f1c71d6ad63171eeeb6a39213ff4aeff7377ac1 /llvm/lib/CodeGen/MachineCombiner.cpp | |
| parent | 35ba467122c5dea1c76d7af586036502c8148834 (diff) | |
| download | llvm-2ef67440fc1e038c52a6bb6d874f34186e6b3b31.zip llvm-2ef67440fc1e038c52a6bb6d874f34186e6b3b31.tar.gz llvm-2ef67440fc1e038c52a6bb6d874f34186e6b3b31.tar.bz2 | |
fix for PR20354 - Miscompile of fabs due to vectorization
This is intended to be the minimal change needed to fix PR20354 ( http://llvm.org/bugs/show_bug.cgi?id=20354 ). The check for a vector operation was wrong; we need to check that the fabs itself is not a vector operation.
This patch will not generate the optimal code. A constant pool load and 'and' op will be generated instead of just returning a value that we can calculate in advance (as we do for the scalar case). I've put a 'TODO' comment for that here and expect to have that patch ready soon.
There is a very similar optimization that we can do in visitFNEG, so I've put another 'TODO' there and expect to have another patch for that too.
llvm-svn: 214670
Diffstat (limited to 'llvm/lib/CodeGen/MachineCombiner.cpp')
0 files changed, 0 insertions, 0 deletions