labath added a comment.

Thanks for looking at this. Could you also tell me what you think of D82864 
<https://reviews.llvm.org/D82864>? That's sort of a more safe/NFC-ish 
alternative to this patch, as it doesn't change behavior -- it  just makes the 
current behavior more obvious. (It's technically possible for the removal of 
the [US]Short and co. accessors to change behavior, but that would require a 
pretty complex set of conditions).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82772



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

Reply via email to