message: Convert code to use early exits in prepraration for future changes. (NFC)
diff: @@ -204,7 +204,9 @@ bool ItaniumABILanguageRuntime::GetDynamicTypeAndAddress(
  // Only a pointer or reference type can have a different dynamic and static
  // type:
  - if (CouldHaveDynamicValue(in_value)) {
  + if (!CouldHaveDynamicValue(in_value))
  + return false;
  +
  // First job, pull out the address at 0 offset from the object.
  AddressType address_type;
  lldb::addr_t original_ptr = in_value.GetPointerValue(&address_type);
  @@ -222,20 +224,23 @@ bool ItaniumABILanguageRuntime::GetDynamicTypeAndAddress(
  const lldb::addr_t vtable_address_point =
  process->ReadPointerFromMemory(original_ptr, error);
  - if (!error.Success() || vtable_address_point == LLDB_INVALID_ADDRESS) {
  + if (!error.Success() || vtable_address_point == LLDB_INVALID_ADDRESS)
  return false;
  - }
  class_type_or_name = GetTypeInfoFromVTableAddress(in_value, original_ptr,
  vtable_address_point);
  - if (class_type_or_name) {
  + if (!class_type_or_name)
  + return false;
  +
  TypeSP type_sp = class_type_or_name.GetTypeSP();
  // There can only be one type with a given name, so we've just found
  // duplicate definitions, and this one will do as well as any other. We
  // don't consider something to have a dynamic type if it is the same as
  // the static type. So compare against the value we were handed.
  - if (type_sp) {
  + if (!type_sp)
  + return true;
  +
  if (ClangASTContext::AreTypesSame(in_value.GetCompilerType(),
  type_sp->GetForwardCompilerType())) {
  // The dynamic type we found was the same type, so we don't have a
  @@ -265,11 +270,6 @@ bool ItaniumABILanguageRuntime::GetDynamicTypeAndAddress(
  }
  return true;
  }
  - }
  - }
  -
  - return class_type_or_name.IsEmpty() == false;
  -}
  TypeAndOrName ItaniumABILanguageRuntime::FixUpDynamicType(
  const TypeAndOrName &type_and_or_name, ValueObject &static_value) {
  