xazax.hun added a comment. In D64448#1577109 <https://reviews.llvm.org/D64448#1577109>, @gribozavr wrote:
> > The tests are currently here > > I think due to their dependency on a standard library, they are not a good > > fit for clang/test/. Where else could I put them? > > Make some mocks that reproduce the salient parts of different libraries, the > coding patterns they use, and check them into clang/test. Those are already there in `clang/test/SemaCXX/attr-gsl-owner-pointer.cpp`. I was thinking about adding the corresponding tests to libc++ as well so we have better chances to get some notifications if a new language feature/implementation trick is introduced. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64448/new/ https://reviews.llvm.org/D64448 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
