I am trying to fix the purely-Clang-related regression tests failures for ConceptClang.<br>I am down to about 10 unexpected failures. But I'm not sure that I'm doing the right thing, since I am having to <br>actually modify clang code that ConceptClang never "touched", and hence that it is not supposed <br>
to modify the behavior of. <br>On the up side, I seem to be fixing some "FIXME" in some of those regression test files, e.g. <br>test/CXX/special/class.temporary/p1.cpp <br>and <br>test/SemaCXX/conversion-function.cpp<br>
<br>To do this, I simply added checks on the results of calls to CheckConstructorAccess()<br> in SemaInit.cpp as in:<br><br>   <b><font size="1"> Sema::AccessResult AcRes = <br></font></b><div style="margin-left: 40px;"><b><font size="1">  S.CheckConstructorAccess(Loc, Constructor, Entity,</font></b><br>
</div><div style="margin-left: 120px;"><b><font size="1">                           Best->FoundDecl.getAccess(), IsExtraneousCopy);</font></b><br></div><b><font size="1">    if (AcRes == Sema::AR_inaccessible)        <br>
        return ExprError();</font></b><br><br>So, my question(s) is(are):<br>1 - Is there any reason that I am overlooking why the implementation was left the way it was?<br>2 - Should the checks be stronger and for <b><font size="1">AcRes != Sema::AR_Accessible</font></b> instead?<br>
3 - Overall, am I doing the right thing at all? And if so, should I submit another patch?<br><br>Sorry if these questions are too basic. I'm still learning the whole framework... :)<br><br>Thanks,<br>-- Larisse.<br><br>