Skip to content

Commit

Permalink
[PowerPC] Set SubRegIndex offset for sub_vsx1/sub_pair1
Browse files Browse the repository at this point in the history
We defined SubRegIndex for 256/512 regs,
but we did not set the offset for higher part,
so the offset of lower and higher part are the same.
This may cause problem in assessing ranges of SubReg,
it is great that this haven't affected any testcases,
but I think we should fix it to avoid hidden bugs in the future.

Reviewed By: bsaleil, #powerpc

Differential Revision: https://reviews.llvm.org/D92864
  • Loading branch information
Jinsong Ji committed Dec 9, 2020
1 parent 02b2c02 commit 45b08c4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions llvm/lib/Target/PowerPC/PPCRegisterInfo.td
Expand Up @@ -17,9 +17,9 @@ def sub_un : SubRegIndex<1, 3>;
def sub_32 : SubRegIndex<32>;
def sub_64 : SubRegIndex<64>;
def sub_vsx0 : SubRegIndex<128>;
def sub_vsx1 : SubRegIndex<128>;
def sub_vsx1 : SubRegIndex<128, 128>;
def sub_pair0 : SubRegIndex<256>;
def sub_pair1 : SubRegIndex<256>;
def sub_pair1 : SubRegIndex<256, 256>;
}


Expand Down

0 comments on commit 45b08c4

Please sign in to comment.