diff options
| author | Craig Topper <craig.topper@intel.com> | 2017-09-28 16:56:36 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2017-09-28 16:56:36 +0000 |
| commit | fd6b8a67fb9e0d5d142e9e2cff5e8c944dc864f3 (patch) | |
| tree | 225de90a261684892c7ab81422b9302308b2bad6 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 71a8cf9f991eb75eb12df76240e68030466d4b1e (diff) | |
| download | llvm-fd6b8a67fb9e0d5d142e9e2cff5e8c944dc864f3.zip llvm-fd6b8a67fb9e0d5d142e9e2cff5e8c944dc864f3.tar.gz llvm-fd6b8a67fb9e0d5d142e9e2cff5e8c944dc864f3.tar.bz2 | |
[X86] Remove dead code from X86ISelDAGToDAG.cpp multiply handling
Summary:
Lowering never creates X86ISD::UMUL for 8-bit types. X86ISD::UMUL8 is used instead. If X86ISD::UMUL 8-bit were ever used it would crash.
DAGCombiner replaces UMUL_LOHI/SMUL_LOHI with a wider MUL and a shift if the type twice as wide is legal. So we should never see i8 UMUL_LOHI/SMUL_LOHI. In fact I think there was a bug in part of the i8 code. Similar is true for i16 though without the bug.
Reviewers: RKSimon, spatel, zvi
Reviewed By: zvi
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D38276
llvm-svn: 314430
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions