[PATCH] D53274: [analyzer][NFC] Fix inconsistencies in AnalyzerOptions

George Karpenkov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 15 15:24:05 PDT 2018


george.karpenkov added a comment.

OK that makes sense to me, just let's be careful not to crash in places where we weren't crashing before.



================
Comment at: lib/StaticAnalyzer/Core/AnalyzerOptions.cpp:354
 
+StringRef AnalyzerOptions::getOptionAsString(Optional<StringRef> &V,
+                                             StringRef Name,
----------------
I'm slightly confused, where is this function called?
Same for the function above?


https://reviews.llvm.org/D53274





More information about the cfe-commits mailing list