<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Oct 21, 2014 at 11:56 AM, Ed Maste <span dir="ltr"><<a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 21 October 2014 14:23, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> No strong preference, but it seems XFAIL is tend to be forgotten. Compared<br>
> to that C++ files are maintained well. So if we can fix this in C++ with a<br>
> TODO, it's better to do that instead of adding a XFAIL to a failing test.<br>
<br>
</span>Note that the cmake HAVE_CXXABI_H test fails on FreeBSD because of<br>
this, so that would need to be addressed first.<br></blockquote><div><br></div><div>It feels to me that HAVE_CXXABI_H is defined properly in a sense because on FreeBSD the header is not actually usable unless you include other header beforehand. Other systems don't have such issue.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I suspect that one of the includes already in<br>
lib/ReaderWriter/ELF/ELFLinkingContext.cpp will bring in the size_t<br>
typedef and so it wouldn't need to be changed.<br>
</blockquote></div><br></div></div>