================
@@ -5357,3 +5358,43 @@ void Target::NotifyBreakpointChanged(
   if (EventTypeHasListeners(Target::eBroadcastBitBreakpointChanged))
     BroadcastEvent(Target::eBroadcastBitBreakpointChanged, breakpoint_data_sp);
 }
+
+llvm::Error
+EvaluateExpressionOptions::SetBooleanLanguageOption(llvm::StringRef 
option_name,
+                                                    bool value) {
+  if (option_name.empty())
+    return llvm::createStringError("Can't set an option with an empty name.");
+
+  GetLanguageOptions().AddBooleanItem(option_name, value);
----------------
jimingham wrote:

There doesn't seem to be any system yet for the Language to validate the 
options you are setting in it.  It would be a stretch to require you to put 
that machinery in place just for this option.  But we should at least have a 
big FIXME here because we really should come up with a way to do that.  And we 
should probably add a note to the documentation that at present we don't 
validate the spelling of options set this way.

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

Reply via email to