bcraig added a comment. In https://reviews.llvm.org/D23279#523449, @alexshap wrote:
> @djasper, @bcraig, @aaron.ballman - many thanks for the comments and > suggestions, > please, let me know if you are waiting for any other changes on my side (for > v0) or if there is anything else i can do to make reviewing easier. I'm not waiting on anything for a v0. Again, just looking at the tests here. I would like to see in a future revision (or this one) a test for the multi-TU support. I would like the test to have one header with a struct definition, and two .c / .cpp files that use the struct definition in a way that would need to be modified. Repository: rL LLVM https://reviews.llvm.org/D23279 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
