[PATCH] D22102: [clang-rename] extend testset

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 11 04:16:49 PDT 2016

klimek added a comment.

// FIXME: <desired behavior to implement>
to tests that do not work yet.

I'd personally not check in the XFAIL tests for now, and just add them when you implement the missing functionality; the problem with XFAIL here is that (due to the offsets involved) they can easily switch into a mode where they fail not because the feature is not implemented, but because the test is incorrect.


More information about the cfe-commits mailing list