<div dir="rtl"><div dir="ltr">For your upcoming patch, some helper functions in DI* should be migrated to the MD <span style="font-size:12.8000001907349px">hierarchy:</span></div><div dir="ltr"><br></div><div dir="ltr">DISubprogram::getFunction()</div><div dir="ltr">DIGlobalVariable::getGlobal()</div><div dir="ltr">*DIDerivedType::getClassType()<br></div><div dir="ltr">*DIDerivedType::getConstant()<br></div><div dir="ltr">(* already in the TODO:)<br></div><div dir="ltr"><br></div><div dir="ltr"><div dir="ltr">Either the new Function *getFunction or the existing one should be renamed.<br></div><div>Thanks!</div><div><br></div><div><br></div></div><div dir="ltr"><br></div></div><div class="gmail_extra"><div dir="ltr"><br><div class="gmail_quote">2015-04-09 21:36 GMT+03:00 Yaron Keren <span dir="ltr"><<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 .8ex;border-left:1px #ccc solid;border-right:1px #ccc solid;padding-left:1ex;padding-right:1ex"><div dir="rtl"><div dir="ltr">Sure, no use in fixing this and another cast in the header file.</div><div dir="ltr"><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-04-09 20:06 GMT+03:00 Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><br>
> On 2015-Apr-09, at 08:38, Yaron Keren <<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>> wrote:<br>
><br>
> This dyn_cast does not seem to be required.<br>
><br>
> Yaron<br>
><br>
</span>> <cast.diff><br>
<br>
For that matter, the `if (N = get()` above it shouldn't be conditional.<br>
<br>
Feel free to commit, but as it happens I'm currently working on deleting<br>
the entire `DIDescriptor` hierarchy so it won't last long anyway ;).<br>
</blockquote></div><br></div>
</div></div></blockquote></div></div></div>