Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(448)

Side by Side Diff: src/IceRegistersMIPS32.h

Issue 2197233002: SubZero: Adding support for all Reg pairs in getI64PairFirst/SecondGPRNum (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Changed as suggested Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | src/IceTargetLoweringMIPS32.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 //===- subzero/src/IceRegistersMIPS32.h - Register information --*- C++ -*-===// 1 //===- subzero/src/IceRegistersMIPS32.h - Register information --*- C++ -*-===//
2 // 2 //
3 // The Subzero Code Generator 3 // The Subzero Code Generator
4 // 4 //
5 // This file is distributed under the University of Illinois Open Source 5 // This file is distributed under the University of Illinois Open Source
6 // License. See LICENSE.TXT for details. 6 // License. See LICENSE.TXT for details.
7 // 7 //
8 //===----------------------------------------------------------------------===// 8 //===----------------------------------------------------------------------===//
9 /// 9 ///
10 /// \file 10 /// \file
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 return FPRRegister(RegNum - Reg_FPR_First); 82 return FPRRegister(RegNum - Reg_FPR_First);
83 } 83 }
84 84
85 static inline bool isFPRReg(RegNumT RegNum) { 85 static inline bool isFPRReg(RegNumT RegNum) {
86 return (int(Reg_FPR_First) <= int(RegNum)) && 86 return (int(Reg_FPR_First) <= int(RegNum)) &&
87 (unsigned(RegNum) <= Reg_FPR_Last); 87 (unsigned(RegNum) <= Reg_FPR_Last);
88 } 88 }
89 89
90 const char *getRegName(RegNumT RegNum); 90 const char *getRegName(RegNumT RegNum);
91 91
92 static inline RegNumT getI64PairFirstGPRNum(RegNumT RegNum) { 92 static inline RegNumT get64PairFirstRegNum(RegNumT RegNum) {
93 // For now it works only for argument register pairs 93 assert(unsigned(RegNum) >= Reg_I64PAIR_First);
94 // TODO(mohit.bhakkad): Change this to support all the register pairs once we 94 assert(unsigned(RegNum) <= Reg_F64PAIR_Last);
95 // have table-driven approach ready 95 if (unsigned(RegNum) >= Reg_F64PAIR_First &&
96 assert(RegNum == Reg_A0A1 || RegNum == Reg_A2A3); 96 unsigned(RegNum) <= Reg_F64PAIR_Last)
97 return (RegNum == RegMIPS32::Reg_A0A1) ? RegMIPS32::Reg_A0 97 return RegNumT::fixme(((RegNum - Reg_F64PAIR_First) * 2) +
98 : RegMIPS32::Reg_A2; 98 unsigned(Reg_FPR_First));
99 if (unsigned(RegNum) >= Reg_I64PAIR_First &&
100 unsigned(RegNum) <= Reg_T8T9)
101 return RegNumT::fixme(((RegNum - Reg_I64PAIR_First) * 2) +
102 unsigned(Reg_V0));
103 return RegMIPS32::Reg_LO;
99 } 104 }
100 105
101 static inline RegNumT getI64PairSecondGPRNum(RegNumT RegNum) { 106 static inline RegNumT get64PairSecondRegNum(RegNumT RegNum) {
102 // For now it works only for argument register pairs 107 assert(unsigned(RegNum) >= Reg_I64PAIR_First);
103 // TODO(mohit.bhakkad): Change this to support all the register pairs once we 108 assert(unsigned(RegNum) <= Reg_F64PAIR_Last);
104 // have table-driven approach ready 109 if (unsigned(RegNum) >= Reg_F64PAIR_First &&
105 assert(RegNum == Reg_A0A1 || RegNum == Reg_A2A3); 110 unsigned(RegNum) <= Reg_F64PAIR_Last)
106 return (RegNum == RegMIPS32::Reg_A0A1) ? RegMIPS32::Reg_A1 111 return RegNumT::fixme(((RegNum - Reg_F64PAIR_First) * 2) +
107 : RegMIPS32::Reg_A3; 112 unsigned(Reg_FPR_First) + 1);
113 if (unsigned(RegNum) >= Reg_I64PAIR_First &&
114 unsigned(RegNum) <= Reg_T8T9)
115 return RegNumT::fixme(((RegNum - Reg_I64PAIR_First) * 2) +
116 unsigned(Reg_V1));
117 return RegMIPS32::Reg_HI;
108 } 118 }
109 119
110 } // end of namespace RegMIPS32 120 } // end of namespace RegMIPS32
111 121
112 // Extend enum RegClass with MIPS32-specific register classes (if any). 122 // Extend enum RegClass with MIPS32-specific register classes (if any).
113 enum RegClassMIPS32 : uint8_t { RCMIPS32_NUM = RC_Target }; 123 enum RegClassMIPS32 : uint8_t { RCMIPS32_NUM = RC_Target };
114 124
115 } // end of namespace MIPS32 125 } // end of namespace MIPS32
116 } // end of namespace Ice 126 } // end of namespace Ice
117 127
118 #endif // SUBZERO_SRC_ICEREGISTERSMIPS32_H 128 #endif // SUBZERO_SRC_ICEREGISTERSMIPS32_H
OLDNEW
« no previous file with comments | « no previous file | src/IceTargetLoweringMIPS32.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698