[PATCH] D26195: Ignore incomplete types when determining whether they are expensive to copy

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 7 15:51:54 PST 2016


alexfh added inline comments.


================
Comment at: clang-tools-extra/trunk/test/clang-tidy/performance-unnecessary-value-param.cpp:241
+
+// Ensure that incomplete types result in an error from the frontend and not a
+// clang-tidy diagnostic about IncompleteType being expensive to copy.
----------------
flx wrote:
> alexfh wrote:
> > Please move this test to a separate test file and revert the RUN line here. 
> Done in https://reviews.llvm.org/D26369.  
Thank you!


Repository:
  rL LLVM

https://reviews.llvm.org/D26195





More information about the cfe-commits mailing list