[PATCH] D20428: Tracking exception specification source locations

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Thu May 19 06:31:36 PDT 2016


aaron.ballman added a comment.

In http://reviews.llvm.org/D20428#434238, @alexfh wrote:

> Full context diff, please.


Pardon my complete ignorance, but how? I generated the diff from svn the usual way, so I assume I've missed some step.

> > I'm not certain of the best way to test this functionality in isolation;

> 

> 

> Same way other locations/ranges are tested in unittests/AST/SourceLocationTest.cpp?


Ah, good point, we have unit tests for this sort of thing. I'll add some unit tests and generate a new patch. Thanks!


http://reviews.llvm.org/D20428





More information about the cfe-commits mailing list