[PATCH] D120586: [Attributor] Add AAAddressSpaceInfo to deduce address spaces

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:11953
+private:
+  // We use an int64_t to make sure the cast from uint32_t does not overflow.
+  int64_t AssumedAddressSpace = NoAddressSpace;
----------------
address spaces only use 24 bits so there's no overflow concern with just u32


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120586