[PATCH] D86635: [Attributor] Add getter of abstract attributes for the manifestation stage

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 26 15:15:44 PDT 2020


jdoerfert added a comment.

Yeah, I'm fine with a member flag. Though we should consider making the thing an enum now, like

  enum {
    SEEDING,
    UPDATE,
    MANIFEST,
  } Phase;

or something like that. Once we have to juggle 2 booleans this might already by easier.


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

https://reviews.llvm.org/D86635



More information about the llvm-commits mailing list