<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 22, 2016 at 11:36 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.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 22 June 2016 at 12:37, Peter Collingbourne <<a href="mailto:peter@pcc.me.uk">peter@pcc.me.uk</a>> wrote:<br>
> pcc added inline comments.<br>
><br>
> ================<br>
> Comment at: include/llvm/IR/Module.h:610<br>
> @@ -609,1 +609,3 @@<br>
><br>
> +  template<class IType, class GIType, class GOType, class MType><br>
> +  class global_object_iterator_t {<br>
> ----------------<br>
> rafael wrote:<br>
>> Why do you need the template?<br>
>><br>
>> MType is always Module, for example.<br>
> It may be either `Module` or `const Module`. The other parameters are similar (see `{,const_}global_object_iterator` below).<br>
<br>
</span>Is that the preferred way to do it? I see that DenseMap has a "bool<br>
IsConst" template parameter. It is a bit more verbose but less<br>
confusing IMHO.<br></blockquote><div><br></div><div>I see. Agreed, I'll make that change then.</div><div><br></div><div>Thanks,</div></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div></div>