================
@@ -1380,6 +1380,12 @@ static llvm::Error CheckScalarOperandsHaveSameType(const 
Scalar &lhs,
   return llvm::Error::success();
 }
 
+static bool IsPotentiallyGenericIntegerOperand(const Scalar &operand,
+                                               size_t address_size) {
+  return address_size != 0 && operand.GetType() == Scalar::e_int &&
+         operand.GetByteSize() == address_size;
+}
----------------
firmiana402 wrote:

I think it would be helpful to add a comment explaining what 
`IsPotentiallyGenericIntegerOperand` means and how it determines whether an 
operand has a generic type.

https://github.com/llvm/llvm-project/pull/218335
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to