[PATCH] D92309: [LegacyPM] Update InversedLastUser on the fly. NFC.

serge via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 30 05:48:54 PST 2020


serge-sans-paille added a comment.

As the state of `LastUser` are closely tied, it would make sense to encapsulate the two members in a class that would maintain the consistency of these two states, so that the two new operations `recordNewUser(Pass, User)` and `updatelastUser(Pass, User)` become explicit ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92309



More information about the llvm-commits mailing list