[cfe-commits] [PATCH] clang/test/Index: 3 of tests require libxml2, following up to r166130.

[jahanian]

On Oct 18, 2012, at 5:45 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:

> Hi Takumi,
> 
> About the patch, adding the feature is not required here.  The correct
> fix is to remove the CommentXMLValid from CHECK lines.  (The test
> checks that there are no CommentXMLInvalid lines, so it is OK).

Yes, I was about to check in this patch. 

- fariborz


> 
> Please tell me if you want me to make this change.
> 
> On Thu, Oct 18, 2012 at 5:30 AM, NAKAMURA Takumi <geek4civic at gmail.com> wrote:
>> FIXME: Could we add the feature 'libxml2' or rename 'xmllint' to one?
> 
> Given my suggestion above, is this still required?
> 
> Dmitri
> 
> -- 
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/