[PATCH] D84186: [clang][cli] Convert Analyzer option string based options to new option parsing system
    Jan Svoboda via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Mon Dec 14 02:51:20 PST 2020
    
    
  
jansvoboda11 added a comment.
Ready for a review.
================
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:287
+static void
+denormalizeString(SmallVectorImpl<const char *> &Args, const char *Spelling,
+                  CompilerInvocation::StringAllocator SA, unsigned, T &&Value) {
----------------
We should keep an eye on the number of instantiations of this function template (and `normalizeStringIntegral`).
If it grows, we can employ the SFINAE trick used for `makeFlagToValueNormalizer`.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84186/new/
https://reviews.llvm.org/D84186
    
    
More information about the cfe-commits
mailing list