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