<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 21, 2021 at 10:58 AM Nigel Perks via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">nigelp-xmos added a comment.<br>
<br>
@echristo Thanks for the comment, Eric. I'm happy to do what is expected. Do you mean that it's a bug not to accept these cases, and we should fix it rather than XFAIL? So the build should not be green from XFAIL but only from a fix in this case?<br></blockquote><div><br></div><div>If they're failing anyhow you can xfail them, but not having any context this looks like it's assumed to not work for a reason other than "the port doesn't work". I'm not sure how to prioritize making the build green here - if it's a recent regression that's going to be fixed imminently that's one thing, if it's a long term xfail that's something else. Either way it probably needs comments and a descriptive commit message :)</div><div><br></div><div>-eric</div></div></div>