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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 23 20:38:47 PDT 2019


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

I think the code below perfectly abstracts out the deleted `unsigned Live : 1;`.

  bool isLive() const { return Partition != 0; }
  void markLive() { Partition = 1; }
  void markDead() { Partition = 0; }

This looks quite good to me, but I am not the proper person to give a final accept.


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