Szelethus marked an inline comment as done.
Szelethus added a comment.
Sure! I'm on my phone, and will be for a little while, can you commit on my
behalf?
================
Comment at: lib/Driver/ToolChains/Clang.cpp:3700
+ // through them.
+ for (size_t Index = 0; Index < Args.size(); ++Index) {
+ if (StringRef(Args.getArgString(Index)).contains("-analyzer-config")) {
----------------
george.karpenkov wrote:
> NoQ wrote:
> > Needs an LLVM-style loop!~ :)
> Shouldn't this be under else- for the previous branch? Otherwise it seems
> that the option would be added twice.
I originally deleted that line, but I lost in somewhere :/
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55823/new/
https://reviews.llvm.org/D55823
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
