phyBrackets added inline comments.
================ Comment at: clang/test/Analysis/bstring_UninitRead.c:67-85 +//===----------------------------------------------------------------------=== +// mempcpy() +//===----------------------------------------------------------------------=== + +#ifdef VARIANT + +#define __mempcpy_chk BUILTIN(__mempcpy_chk) ---------------- phyBrackets wrote: > steakhal wrote: > > I don't think you should copy these as well. > > The matching logic is already tested in `bstring.c`. > > > > That being said, a single `RUN` line should be enough in this file. > I'm not able to run test successfully with single RUN , > ``` > // RUN: -analyzer-checker=alpha.unix.cstring.UninitializedRead > > ``` wait , I think I got this! Thanks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120489/new/ https://reviews.llvm.org/D120489 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
