[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 03:10:59 PST 2021


dfukalov added a subscriber: rampitec.
dfukalov added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp:44-80
 // These arrays are indexed by address space value enum elements 0 ... to 7
+//                 Flat     | Global   | Region   | Group    | Constant | Private | Cnst 32b | BFPtr
+// ----------------------------------------------------------------------------------------------------
+// Flat            MayAlias | MayAlias | NoAlias  | MayAlias | MayAlias | MayAlias| MayAlias | MayAlias
+// Global          MayAlias | MayAlias | NoAlias  | NoAlias  | MayAlias | NoAlias | MayAlias | MayAlias
+// Region          NoAlias  | NoAlias  | MayAlias | NoAlias  | NoAlias  | NoAlias | NoAlias  | NoAlias
+// Group           MayAlias | NoAlias  | NoAlias  | MayAlias | NoAlias  | NoAlias | NoAlias  | NoAlias
----------------
@rampitec, @arsenm, would you please take a look to this 'typography'? I moved out table comments since the table width increased significantly with `AliasResult::`s


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