<div dir="ltr">Hi Christof,<div><br></div><div>I committed this on your behalf in r250033. Now might be a good time to ask for SVN commit access.</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 12 Oct 2015 at 13:02 Christof Douma via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've opened <a href="https://llvm.org/bugs/show_bug.cgi?id=25145" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=25145</a> for adding the missing instructions.<br>
Can I ask somebody to commit this, as I don't have commit rights yet.<br>
<br>
Thanks,<br>
Christof<br>
<br>
> -----Original Message-----<br>
> From: Andrew Trick [mailto:<a href="mailto:atrick@apple.com" target="_blank">atrick@apple.com</a>]<br>
> Sent: 09 October 2015 19:23<br>
> To: Christof Douma<br>
> Cc: <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>; Arnold Schwaighofer; Matthias Braun<br>
> Subject: Re: [PATCH] Marked Swift Machine Scheduler Model as incomplete<br>
><br>
> Hi Christof,<br>
><br>
> The change is fine with me as long as you file a bug to get those instructions<br>
> added to the model.<br>
><br>
> Andy<br>
><br>
> > On Oct 9, 2015, at 8:53 AM, Christof Douma <<a href="mailto:Christof.Douma@arm.com" target="_blank">Christof.Douma@arm.com</a>><br>
> wrote:<br>
> ><br>
> > Hi.<br>
> ><br>
> > I've got a 1-line change to the SwiftModel to mark the model as<br>
> incomplete.<br>
> > Can I get a review on this?<br>
> ><br>
> > Thanks,<br>
> > Christof<br>
> > ------------<br>
> > Marked Swift Machine Scheduler Model as incomplete<br>
> ><br>
> > Swift Machine Scheduler Model is incomplete. There are instructions<br>
> > missing which can trigger the "incomplete machine model" abort. This<br>
> > was observed when a downstream SchedMachineModel was added to the<br>
> ARM<br>
> > target.<br>
> ><br>
> > Change-Id: Ie43cbc8d529b6a1257a5d7b1e69e6f40d504b842<br>
> > ---<br>
> > lib/Target/ARM/ARMScheduleSwift.td | 1 +<br>
> > 1 file changed, 1 insertion(+)<br>
> ><br>
> > diff --git a/lib/Target/ARM/ARMScheduleSwift.td<br>
> > b/lib/Target/ARM/ARMScheduleSwift.td<br>
> > index 6f5740f..64b477d 100644<br>
> > --- a/lib/Target/ARM/ARMScheduleSwift.td<br>
> > +++ b/lib/Target/ARM/ARMScheduleSwift.td<br>
> > @@ -43,6 +43,7 @@ def SwiftModel : SchedMachineModel {<br>
> >   let MicroOpBufferSize = 45; // Based on NEON renamed registers.<br>
> >   let LoadLatency = 3;<br>
> >   let MispredictPenalty = 14; // A branch direction mispredict.<br>
> > +  let CompleteModel = 0;    // FIXME: Remove if all instructions are<br>
> > covered.<br>
> > }<br>
> ><br>
> > // Swift predicates.<br>
> > --<br>
> > 1.8.3<br>
> ><br>
> ><br>
<br>
<br>
-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.<br>
<br>
ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2557590<br>
ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2548782<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>