<div dir="ltr">Hi All,<div><br></div><div>Alexey, Takumi: Sorry, I missed that fix. Evidently I need to expand my commit filters.</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 style="font-size:13px">I think you can solve the problem the same way I suggested that Chandler do it: make the derived error classes responsible for providing this symbol, and have classID() return &ThisErrT::ID.</span></blockquote><div><span style="font-size:13px"><br></span></div><div>Yep - this is the best way to fix this. I'll make the changes.</div><div><span style="font-size:13px"><br></span></div><div>Cheers,</div><div>Lang.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 24, 2016 at 4:33 PM, Reid Kleckner via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Thu, Mar 24, 2016 at 4:31 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Pretty unfortunate additional boilerplate (perhaps moreso in the pass manager than in the error categories, not sure which we'll end up with more of in the long run), but if that's the nearest we can do... </div></div></div></div></blockquote><div><br></div></span><div>Sure, but it's normal, character-building, C++ boilerplate. We still prototype our methods in header files like cavemen, rather than making every function inline.</div></div></div></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>