<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 11, 2013 at 10:04 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank" class="cremed">hfinkel@anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">----- Original Message -----<br>

><br>
><br>
><br>
><br>
> On Sat, Nov 9, 2013 at 5:44 AM, Hal Finkel < <a href="mailto:hfinkel@anl.gov" class="cremed">hfinkel@anl.gov</a> > wrote:<br>
><br>
><br>
> Okay, I've attached a patch that adds this into SimplifyLibCalls. In<br>
> this incarnation, I've added the functionality at the top-level of<br>
> SimplifyLibCalls itself (which makes it different from all of the<br>
> other simplifications). My rationale for this is that adding the<br>
> cold hint should not depend on the builtin status of the functions.<br>
> I'm not sure how much I like this. Thoughts?<br>
> I think it should depend on the builtin status of the functions. If<br>
> we're in a freestanding environment, there is no reason to believe<br>
> that the meanings for the various file descriptors continue to hold.<br>
<br>
</div></div>Chandler, Meador, et al.,<br>
<br>
Thanks for your feedback! I've attached a new version (which makes this look much like all of the other optimizations). Please review.<br></blockquote><div><br></div><div>While I deeply question the fundamental design here, I suspect your patch should follow it.</div>
<div> </div><div><div>+    ErrorReportingOpt ER(/* StreamArg = */ 0);</div><div>+    (void) ER.callOptimizer(Callee, CI, B);</div><div>+</div></div><div><br></div><div>The way the rest of the simplify lib calls works is for specific lib calls to subclass a more more specialized base class to provide delegated optimizations, not to build an instance of the class...</div>
<div><br></div><div>Bah, maybe just put this in a couple of static function and use a more boring design? Seems simpler.</div><div><br></div><div>-Chandler</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<span class=""><font color="#888888"><br>
 -Hal<br>
</font></span><div class=""><div class="h5"><br>
--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</div></div></blockquote></div><br></div></div>