================
@@ -559,6 +559,34 @@ Interpreter::Visit(const UnaryOpNode &node) {
     }
     return operand;
   }
+  case UnaryOpKind::Not: {
+    llvm::Expected<lldb::ValueObjectSP> conv_op =
+        UnaryConversion(operand, node.GetLocation());
----------------
kuilpd wrote:

> Should we dereference reference operands before calling `UnaryConversion` 
> here? The existing unary `+` and unary `-` branches already handle this by 
> checking `IsReferenceType()` and calling `Dereference()` before the unary 
> conversion.

You're right, thank you for noticing!

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

Reply via email to