<div dir="ltr">Oops! I just committed the patch of moving GlobalMerge pass to CodeGen.<div><br></div><div>Eric, sorry I didn't see your email early. If you are not happy with this change, I can revert both of the patch adding macro INITIALIZE_TM_PASS and this patch.</div>
<div><br></div><div>For me, I think Candler's explanation is clear, and at least at this moment, there isn't pass depending on TargetMachine under lib/Transform.</div><div><br></div><div>Thanks,</div><div>-Jiangning</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">2014-06-14 6:46 GMT+08:00 Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
Sent from my iPhone<br>
<div class=""><br>
> On Jun 13, 2014, at 18:32, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
><br>
>> On Fri, Jun 13, 2014 at 3:28 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
>><br>
>> On Fri, Jun 13, 2014 at 11:02 PM, Rafael Avila de Espindola<br>
>> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>>><br>
>>>> On Jun 13, 2014, at 17:26, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
>>>><br>
>>>> Ack. This is a terrible idea. (As there's a thread on it)<br>
>>>><br>
>>>> But I'm not sure what the current status is here, did this patch get<br>
>>>> reverted?<br>
>>><br>
>>><br>
>>> I don't think so. My understanding is that Jiangning Liu is working to<br>
>>> move the pass to lib/CodeGen. Would that make it ok for you or there are<br>
>>> other issues besides the layering problem?<br>
>><br>
>><br>
>> I'm not sure the nature of the problem Eric is hitting, but my problems are<br>
>> well addressed by the patch already mailed out on this thread. For me, that<br>
>> would be a step in the right directoin.<br>
><br>
> Oh, I'm just going to have to rewrite it to avoid taking a TM, but<br>
> this isn't making it any worse afaict.<br>
><br>
<br>
</div>Because it is a module pass and you are changing each function to have a different one?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> -eric<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>