RedDocMD marked an inline comment as done. RedDocMD added inline comments.
================ Comment at: clang/test/Analysis/smart-ptr-text-output.cpp:3 +// RUN: -analyzer-checker=core,cplusplus.Move,alpha.cplusplus.SmartPtr\ +// RUN: -analyzer-config cplusplus.SmartPtrModeling:ModelSmartPtrDereference=true\ +// RUN: -analyzer-output=text -std=c++20 %s -verify=expected ---------------- NoQ wrote: > RedDocMD wrote: > > NoQ wrote: > > > Could you double-check that this flag correctly disables all the newly > > > introduced modeling so that it wasn't accidentally enabled for all users > > > before it's ready? > > Yup. There are 133 notes and warnings (just search for "// expected-" and > > see the count). And on setting the flag to `false`, there are 133 errors. > > So it works. > I mean, specifically the modeling added by this patch? I see you wrote an > entire checker callback that doesn't seem to have an analyzer-config option > check anywhere in it. Simply having different results isn't sufficient to > verify this. Oh I see. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103750/new/ https://reviews.llvm.org/D103750 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
