Merge pull request #16092 from CarolEidt/RefactorRefBuilding

Refactor RefPosition and Interval Building

diff --cc src/jit/lsraarm64.cpp
index 8dd299f,990f39d..82ca013
--- 1/src/jit/lsraarm64.cpp
--- 2/src/jit/lsraarm64.cpp
+++ b/src/jit/lsraarm64.cpp
@@@ -982,9 -910,8 +910,9 @@@ void LinearScan::BuildSIMD(GenTreeSIMD
  #endif // FEATURE_SIMD
  
  #ifdef FEATURE_HW_INTRINSICS
 +#include "hwintrinsicArm64.h"
  //------------------------------------------------------------------------
- // TreeNodeInfoInitHWIntrinsic: Set the NodeInfo for a GT_HWIntrinsic tree.
+ // BuildHWIntrinsic: Set the NodeInfo for a GT_HWIntrinsic tree.
  //
  // Arguments:
  //    tree       - The GT_HWIntrinsic node of interest
@@@ -992,47 -919,14 +920,48 @@@
  // Return Value:
  //    None.
  
- void LinearScan::TreeNodeInfoInitHWIntrinsic(GenTreeHWIntrinsic* intrinsicTree, TreeNodeInfo* info)
+ void LinearScan::BuildHWIntrinsic(GenTreeHWIntrinsic* intrinsicTree)
  {
+     TreeNodeInfo*  info        = currentNodeInfo;
      NamedIntrinsic intrinsicID = intrinsicTree->gtHWIntrinsicId;
 -    info->srcCount += GetOperandInfo(intrinsicTree->gtOp.gtOp1);
 -    if (intrinsicTree->gtGetOp2IfPresent() != nullptr)
 +
 +    GenTreeArgList* argList = nullptr;
 +    GenTree*        op1     = intrinsicTree->gtOp.gtOp1;
 +    GenTree*        op2     = intrinsicTree->gtOp.gtOp2;
 +
 +    if (op1->OperIs(GT_LIST))
      {
 -        info->srcCount += GetOperandInfo(intrinsicTree->gtOp.gtOp2);
 +        argList = op1->AsArgList();
 +        op1     = argList->Current();
 +        op2     = argList->Rest()->Current();
 +
 +        for (GenTreeArgList* list = argList; list != nullptr; list = list->Rest())
 +        {
 +            info->srcCount += GetOperandInfo(list->Current());
 +        }
 +    }
 +    else
 +    {
 +        info->srcCount += GetOperandInfo(op1);
 +        if (op2 != nullptr)
 +        {
 +            info->srcCount += GetOperandInfo(op2);
 +        }
 +    }
 +
 +    switch (compiler->getHWIntrinsicInfo(intrinsicID).form)
 +    {
 +        case HWIntrinsicInfo::SimdExtractOp:
 +            if (!op2->isContained())
 +            {
 +                // We need a temp to create a switch table
 +                info->internalIntCount = 1;
 +                info->setInternalCandidates(this, allRegs(TYP_INT));
 +            }
 +            break;
 +
 +        default:
 +            break;
      }
  }
  #endif

diff --cc src/jit/lsraxarch.cpp
Simple merge