Skip to content

Commit 2ca6d8b

Browse files
yoneymeta-codesync[bot]
authored andcommitted
Fix operand constraints for zero-operand HIR instructions
Summary: `MakeList`, `MakeTuple`, and `MakeCheckedList` now match their zero-operand definitions. Reviewed By: alexmalyshev Differential Revision: D110335478 fbshipit-source-id: 006bdf638d23099be2b1f09c32129df795cde712
1 parent c2f36ab commit 2ca6d8b

1 file changed

Lines changed: 3 additions & 8 deletions

File tree

cinderx/Jit/hir/hir.h

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2922,7 +2922,7 @@ class INSTR_CLASS(LoadArg, (), HasOutput, Operands<0>) {
29222922
};
29232923

29242924
// Allocate an empty list object
2925-
class INSTR_CLASS(MakeList, (TObject), HasOutput, Operands<0>, DeoptBase) {
2925+
class INSTR_CLASS(MakeList, (), HasOutput, Operands<0>, DeoptBase) {
29262926
public:
29272927
MakeList(Register* dst, size_t nvalues, const FrameState& frame)
29282928
: InstrT(dst, frame), nvalues_(nvalues) {}
@@ -2936,7 +2936,7 @@ class INSTR_CLASS(MakeList, (TObject), HasOutput, Operands<0>, DeoptBase) {
29362936
};
29372937

29382938
// Allocate an empty tuple object
2939-
class INSTR_CLASS(MakeTuple, (TObject), HasOutput, Operands<0>, DeoptBase) {
2939+
class INSTR_CLASS(MakeTuple, (), HasOutput, Operands<0>, DeoptBase) {
29402940
public:
29412941
MakeTuple(Register* dst, size_t nvalues, const FrameState& frame)
29422942
: InstrT(dst, frame), nvalues_(nvalues) {}
@@ -3185,12 +3185,7 @@ class INSTR_CLASS(MakeCheckedDict, (), HasOutput, Operands<0>, DeoptBase) {
31853185
};
31863186

31873187
// Allocate and fill a CheckedList object with the given operands
3188-
class INSTR_CLASS(
3189-
MakeCheckedList,
3190-
(TObject),
3191-
HasOutput,
3192-
Operands<0>,
3193-
DeoptBase) {
3188+
class INSTR_CLASS(MakeCheckedList, (), HasOutput, Operands<0>, DeoptBase) {
31943189
public:
31953190
MakeCheckedList(
31963191
Register* dst,

0 commit comments

Comments
 (0)