alexfh added inline comments.
Herald added a subscriber: jdoerfert.
Herald added a project: LLVM.


================
Comment at: cfe/trunk/lib/StaticAnalyzer/Checkers/MmapWriteExecChecker.cpp:31-33
+  static int ProtWrite;
+  static int ProtExec;
+  static int ProtRead;
----------------
Can these be non-static members instead? These are one of just a few uses of 
static fields in Clang.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D42645



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to