[PATCH] D53280: [analyzer] Emit an error for invalid -analyzer-config inputs

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 29 16:16:32 PST 2018


Szelethus marked 2 inline comments as done.
Szelethus added a comment.

Thanks!



================
Comment at: lib/Frontend/CompilerInvocation.cpp:363
+      A->claim();
       Opts.Config[key] = val;
     }
----------------
NoQ wrote:
> Maybe we can eventually turn this into an array and address by index that's auto-generated from the options table.
> 
> But that's pretty unimportant; even though we can query options pretty often, i don't think they are a performance bottleneck.
Actually, `ConfigTable` will probably end up on the chopping board by the end of the refactoring.


Repository:
  rC Clang

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

https://reviews.llvm.org/D53280





More information about the cfe-commits mailing list