[PATCH] D98027: [NFC][AA] Convert AliasResult to scoped enumeration.

Daniil Fukalov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 5 09:46:40 PST 2021


dfukalov added a comment.

No, I don't have a patch at the moment, but will upload one since you're generally ok with this step.

I thought to convert the enum to a something like

  class AliasResult {
  public:
    int16_t Offset : 14;
    uint8_t EnumStorage : 2;
    ...
  public:
    enum {...};
    ...
  }

So after that access to the enum values needs `AliasResult::` and I decided to split this NFC part away.
And using scoped enum allowed to find all uses.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98027



More information about the llvm-commits mailing list