<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 8, 2015 at 1:57 PM, Sergey Dmitrouk <span dir="ltr"><<a href="mailto:sdmitrouk@accesssoftek.com" target="_blank">sdmitrouk@accesssoftek.com</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 Fri, May 08, 2015 at 01:00:51PM -0700, David Blaikie wrote:<br>
> How's about we just overload, then? (& assert that the right function is<br>
> called for the right kinds of things? (ie: assert that the 3 arg (the one<br>
> with DebugLoc) is called only for Constant/ConstantFP, and the 2 arg is<br>
> called for non-Constant/ConstantFP? Would that make sense))<br>
<br>
</span>Not really :)  If we knew that this is neither Constant nor ConstantFP,<br>
we could just call CSEMap directly.  The method was added to omit<br>
checking opcode in all 40 places (only several callers have opcodes<br>
nearby and could make correct decision which function to call, others<br>
take opcode as one of their parameters).<br></blockquote><div><br>OK - can we assert on one side, perhaps? *shrug* Anyway, whatever works/suits.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Sergey<br>
</font></span></blockquote></div><br></div></div>