Minor changes for code quality and clarity

Author: JamesChesterman

llvm/include/llvm/CodeGen/SelectionDAGNodes.h

@@ -2999,9 +2999,7 @@ class MaskedHistogramSDNode : public MaskedGatherScatterSDNode {
                         EVT MemVT, MachineMemOperand *MMO,
                         ISD::MemIndexType IndexType)
       : MaskedGatherScatterSDNode(ISD::EXPERIMENTAL_VECTOR_HISTOGRAM, Order, DL,
-                                  VTs, MemVT, MMO, IndexType) {
-    LSBaseSDNodeBits.AddressingMode = IndexType;
-  }
+                                  VTs, MemVT, MMO, IndexType) {}
 
   ISD::MemIndexType getIndexType() const {
     return static_cast<ISD::MemIndexType>(LSBaseSDNodeBits.AddressingMode);

llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

@@ -12378,7 +12378,6 @@ SDValue DAGCombiner::visitMHISTOGRAM(SDNode *N) {
                                   MMO, IndexType);
   }
   EVT DataVT = Index.getValueType();
-  DataVT.changeVectorElementType(Inc.getValueType());
   if (refineIndexType(Index, IndexType, DataVT, DAG)) {
     return DAG.getMaskedHistogram(DAG.getVTList(MVT::Other), MemVT, DL, Ops,
                                   MMO, IndexType);

llvm/lib/Target/AArch64/AArch64ISelLowering.cpp

@@ -24092,9 +24092,8 @@ static SDValue performMaskedGatherScatterCombine(
   SDValue BasePtr = MGS->getBasePtr();
   ISD::MemIndexType IndexType = MGS->getIndexType();
 
-  if (!findMoreOptimalIndexType(MGS, BasePtr, Index, DAG)) {
+  if (!findMoreOptimalIndexType(MGS, BasePtr, Index, DAG))
     return SDValue();
-  }
 
   // Here we catch such cases early and change MGATHER's IndexType to allow
   // the use of an Index that's more legalisation friendly.