jasonmolenda added a comment.

In D78077#1980116 <https://reviews.llvm.org/D78077#1980116>, @clayborg wrote:

> If you test worked, then there is something wrong with this test? See inline 
> comment for copy and paste error


Thanks for catching that.  The test definitely fails without the patch, and 
works with.  lldb is using that sp_reg_info to re-set the m_fp_is_cfa ivar, 
which my test clearly does not exercise if it is wrong.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78077/new/

https://reviews.llvm.org/D78077



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to