[PATCH] D24821: [ASTMatcher] Add isStaticStorageClass matcher for varDecl and functionDecl.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 26 09:48:16 PDT 2016


hokein added inline comments.

================
Comment at: cfe/trunk/include/clang/ASTMatchers/ASTMatchers.h:3391
@@ +3390,3 @@
+/// \brief Matches variable/function declarations that have static storage class
+/// (with "static" key word) written in the source.
+///
----------------
aaron.ballman wrote:
> It would be nice if you could clarify how this differs from the storage duration. This is a bit confusing because we already have `hasStaticStorageDuration()` for variables.
Good point! Will make a follow-up patch to do it.


Repository:
  rL LLVM

https://reviews.llvm.org/D24821





More information about the cfe-commits mailing list