[Lldb-commits] green dragon testsuite crash on TestCoroutineHandle.py from coroutines formatter changes, temporarily revert?

[Jason Molenda via lldb-commits]

Hi Adrian, the green dragon Incremental CI bot has been failing the past couple of days after the changes for the coroutines formatter, most directly   https://reviews.llvm.org/D132815 landed - TestCoroutineHandle.py results in a segfault on Darwin systems (both on the CI and on my mac desktop) consistently.   https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/

It's crashing in formatters::StdlibCoroutineHandleSyntheticFrontEnd where you're doing

```
  // Add the `promise` member. We intentionally add `promise` as a pointer type
  // instead of a value type, and don't automatically dereference this pointer.
  // We do so to avoid potential very deep recursion in case there is a cycle in
  // formed between `std::coroutine_handle`s and their promises.
  lldb::ValueObjectSP promise = CreateValueObjectFromAddress(
      "promise", frame_ptr_addr + 2 * ptr_size, exe_ctx, promise_type);
  Status error;
  lldb::ValueObjectSP promisePtr = promise->AddressOf(error);
```

and the promise_type dose not have a CompilerType so promisePtr is nullptr and we crash on the method call to AddressOf. I looked briefly, but this isn't a part of lldb I know well and I'm not sure the nature of the problem.

It's a long weekend in the US, so if you have a suggestion for addressing this that'd be great, or I can roll back the changes necessary to get the bot clean later today and we can look at this next week.  

Thanks!