[llvm] [SelectionDAG] use HandleSDNode instead of SDValue during SelectInlineAsmMemoryOperands (PR #85081)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 8 22:31:01 PDT 2024


================
@@ -2111,24 +2111,27 @@ bool SelectionDAGISel::CheckOrMask(SDValue LHS, ConstantSDNode *RHS,
 /// by tblgen.  Others should not call it.
 void SelectionDAGISel::SelectInlineAsmMemoryOperands(std::vector<SDValue> &Ops,
                                                      const SDLoc &DL) {
-  std::vector<SDValue> InOps;
-  std::swap(InOps, Ops);
+  // change the vector of SDValue into a list of SDNodeHandle for x86 might call
----------------
topperc wrote:

This should be a complete sentence starting with a capital letter and ending with a period.

https://github.com/llvm/llvm-project/pull/85081


More information about the llvm-commits mailing list