mstorsjo added a comment.

In D128268#3601104 <https://reviews.llvm.org/D128268#3601104>, @DavidSpickett 
wrote:

> LGTM. I like the ifs the way I suggested but up to you, either way it's not 
> the easiest logic to parse.

Oh, sorry, I forgot about that comment. Yes, that does indeed look even nicer, 
I'll try to add that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128268

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

Reply via email to