[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