diff options
| author | Craig Topper <craig.topper@intel.com> | 2019-02-23 08:34:10 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2019-02-23 08:34:10 +0000 |
| commit | 75afc0105c089171f9d85d59038617fb222c38cd (patch) | |
| tree | 568cdcd8058cc08c4fcaf8865a918edc582ffcee /libcxx/test | |
| parent | 6f1268a4ccad108bd478d520e2cd647d709a297b (diff) | |
| download | bcm5719-llvm-75afc0105c089171f9d85d59038617fb222c38cd.tar.gz bcm5719-llvm-75afc0105c089171f9d85d59038617fb222c38cd.zip | |
[X86] Sign extend the 8-bit immediate when commuting blend instructions to match isel.
Conversion from ConstantSDNode to MachineInstr sign extends immediates from their APInt representation to int64_t.
This commit makes sure we do the same for commuting. The tests changes show how this improves CSE. This issue was made worse by the MachineCSE using commuteInstruction to undo a commute. So we virtually guarantee the sign extend from isel would be lost.
The improved CSE also occurred with r354363, but that was reverted. I'm working to undo the revert, but wanted to get this fix in while it was easy to see the results.
llvm-svn: 354724
Diffstat (limited to 'libcxx/test')
0 files changed, 0 insertions, 0 deletions
