[cfe-dev] Patch for UCNs in string literals
Eli Friedman
eli.friedman at gmail.com
Sat Jul 4 23:57:35 PDT 2009
On Sat, Jul 4, 2009 at 10:16 PM, James Porter<porterj at alum.rit.edu> wrote:
> Eli Friedman wrote:
>> Please just include the subset of tests that currently passes; we can
>> add additional tests as the issues get fixed.
>
> I was under the impression based on the website that XFAIL'd tests were
> encouraged:
>
> http://clang.llvm.org/OpenProjects.html
>
> "Pick a paragraph, write a few tests, and see if they work! Even if
> they don't we'd still like the new tests (with XFAIL'd) so that we
> know what to fix."
A patch should be accompanied by a non-XFAIL'd test for each thing it
fixes; otherwise, the tests won't detect regressions. I haven't
thought through what that means for the C++ tests; perhaps splitting
them into p2a/p2b/p2c/etc. would be appropriate.
-Eli
More information about the cfe-dev
mailing list