[PATCH] D22450: [CFLAA] Add interprocerual analysis support to CFLAndersAliasAnalysis

Jia Chen via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 19 09:47:12 PDT 2016


grievejia added inline comments.

================
Comment at: lib/Analysis/CFLAndersAliasAnalysis.cpp:99
@@ +98,3 @@
+typedef std::bitset<8> StateSet;
+LLVM_CONSTEXPR StateSet ReadOnlyStateMask =
+    (1 << static_cast<uint8_t>(MatchState::FlowFromReadOnly)) |
----------------
george.burgess.iv wrote:
> Is there a reason these aren't `static`?
They are defined inside anonymous namespace so I assume they're implicitly static? We did the same thing for those StratifiedIndices.


https://reviews.llvm.org/D22450





More information about the llvm-commits mailing list