[PATCH] D60353: ELF: Add basic partition data structures and behaviours.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 20 00:11:01 PDT 2019


ruiu added a comment.

In D60353#1506522 <https://reviews.llvm.org/D60353#1506522>, @MaskRay wrote:

> > although I'd like to somehow continue using the term "Live" instead of Partition is zero" to explain liveness in code and comments, though.
>
> I think this is fine... After all all `S->Live` uses have been changed to `S->isLive()`


Yeah, but killing it is `S->Partition = 0` which isn't very intuitive.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60353





More information about the llvm-commits mailing list