Delete some unused functions & one define (#53130)
authorSingleAccretion <62474226+SingleAccretion@users.noreply.github.com>
Wed, 26 May 2021 02:44:12 +0000 (05:44 +0300)
committerGitHub <noreply@github.com>
Wed, 26 May 2021 02:44:12 +0000 (19:44 -0700)
src/coreclr/jit/gentree.h
src/coreclr/jit/lowerxarch.cpp
src/coreclr/jit/vartype.h

index 669a798..10e61bb 100644 (file)
@@ -1779,12 +1779,6 @@ public:
         return OperIsAnyList(gtOper);
     }
 
-    inline GenTree* MoveNext();
-
-    inline GenTree* Current();
-
-    inline GenTree** pCurrent();
-
     inline GenTree* gtGetOp1() const;
 
     // Directly return op2. Asserts the node is binary. Might return nullptr if the binary node allows
@@ -7197,12 +7191,6 @@ inline bool GenTree::IsBoxedValue()
     return (gtOper == GT_BOX) && (gtFlags & GTF_BOX_VALUE);
 }
 
-inline GenTree* GenTree::MoveNext()
-{
-    assert(OperIsAnyList());
-    return AsOp()->gtOp2;
-}
-
 #ifdef DEBUG
 //------------------------------------------------------------------------
 // IsValidCallArgument: Given an GenTree node that represents an argument
@@ -7251,18 +7239,6 @@ inline bool GenTree::IsValidCallArgument()
 }
 #endif // DEBUG
 
-inline GenTree* GenTree::Current()
-{
-    assert(OperIsAnyList());
-    return AsOp()->gtOp1;
-}
-
-inline GenTree** GenTree::pCurrent()
-{
-    assert(OperIsAnyList());
-    return &(AsOp()->gtOp1);
-}
-
 inline GenTree* GenTree::gtGetOp1() const
 {
     return AsOp()->gtOp1;
index d7e7746..4acfd81 100644 (file)
@@ -6360,8 +6360,8 @@ void Lowering::ContainCheckHWIntrinsic(GenTreeHWIntrinsic* node)
                                     MakeSrcContained(node, op1);
                                     // MultiplyNoFlags is a Commutative operation, so swap the first two operands here
                                     // to make the containment checks in codegen significantly simpler
-                                    *(originalArgList->pCurrent())         = op2;
-                                    *(originalArgList->Rest()->pCurrent()) = op1;
+                                    originalArgList->Current()         = op2;
+                                    originalArgList->Rest()->Current() = op1;
                                 }
                                 else if (supportsRegOptional)
                                 {
index f1b134d..18cee73 100644 (file)
@@ -35,11 +35,9 @@ enum var_types : BYTE
 #ifdef TARGET_64BIT
 #define TYP_I_IMPL TYP_LONG
 #define TYP_U_IMPL TYP_ULONG
-#define TYPE_REF_IIM TYPE_REF_LNG
 #else
 #define TYP_I_IMPL TYP_INT
 #define TYP_U_IMPL TYP_UINT
-#define TYPE_REF_IIM TYPE_REF_INT
 #ifdef _PREFAST_
 // We silence this in the 32-bit build because for portability, we like to have asserts like this:
 // assert(op2->gtType == TYP_INT || op2->gtType == TYP_I_IMPL);