gchatelet marked 2 inline comments as done. gchatelet added inline comments.
================ Comment at: clang/test/Sema/builtins-memcpy-inline.c:7 +#warning defined as expected +// expected-warning@-1 {{defined as expected}} +#endif ---------------- gchatelet wrote: > efriedma wrote: > > melver wrote: > > > It appears that the expected-warning check here is guarded by the #if as > > > well. Moving it after the #endif results in a failing test. > > > > > > I noticed this as I was trying to use > > > __has_feature(__builtin_memcpy_inline), but it somehow does not work, > > > even though the compiler clearly supports __builtin_memcpy_inline. > > > > > > Any idea what's wrong with the __has_feature test? > > > > > > Thanks! > > Should be `__has_builtin`, I think? __has_feature only applies to features > > defined in clang/include/clang/Basic/Features.def. > Thx for noticing. I'll send a patch to update the test. Submitted as rG12c8e3632edda3bc640867ba326b52b5b2220a57 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73543/new/ https://reviews.llvm.org/D73543 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits