diff options
| author | Hal Finkel <hfinkel@anl.gov> | 2015-09-03 21:12:15 +0000 |
|---|---|---|
| committer | Hal Finkel <hfinkel@anl.gov> | 2015-09-03 21:12:15 +0000 |
| commit | 99d95328d6af9b99a607aec6ece9a950c38c8913 (patch) | |
| tree | 3c12e64e441eb723eb7f8a620135e33914c16e69 /llvm/lib/Target/PowerPC/PPCISelLowering.cpp | |
| parent | c079df09a6c38a88ee4f921857682413350ff734 (diff) | |
| download | bcm5719-llvm-99d95328d6af9b99a607aec6ece9a950c38c8913.tar.gz bcm5719-llvm-99d95328d6af9b99a607aec6ece9a950c38c8913.zip | |
[PowerPC] Compute the MMO offset for an unaligned load with signed arithmetic
If you compute the MMO offset using unsigned arithmetic, you end up with a
large positive offset instead of a small negative one. In theory, this could
cause bad instruction-scheduling decisions later.
I noticed this by inspection from the debug output, and using that for the
regression test is the best I can do right now.
llvm-svn: 246805
Diffstat (limited to 'llvm/lib/Target/PowerPC/PPCISelLowering.cpp')
| -rw-r--r-- | llvm/lib/Target/PowerPC/PPCISelLowering.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Target/PowerPC/PPCISelLowering.cpp b/llvm/lib/Target/PowerPC/PPCISelLowering.cpp index 40a80d62113..116ecb4537e 100644 --- a/llvm/lib/Target/PowerPC/PPCISelLowering.cpp +++ b/llvm/lib/Target/PowerPC/PPCISelLowering.cpp @@ -10302,7 +10302,8 @@ SDValue PPCTargetLowering::PerformDAGCombine(SDNode *N, // original unaligned load. MachineFunction &MF = DAG.getMachineFunction(); MachineMemOperand *BaseMMO = - MF.getMachineMemOperand(LD->getMemOperand(), -MemVT.getStoreSize()+1, + MF.getMachineMemOperand(LD->getMemOperand(), + -(long)MemVT.getStoreSize()+1, 2*MemVT.getStoreSize()-1); // Create the new base load. |
