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

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 29 11:57:18 PST 2018


NoQ accepted this revision.
NoQ added a comment.

Thanks, this looks great!



================
Comment at: lib/Frontend/CompilerInvocation.cpp:363
+      A->claim();
       Opts.Config[key] = val;
     }
----------------
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.


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