<div dir="ltr">Ok, final draft, I think the patch is good to go.<div><br></div><div style>I now have reverted most of the C++-ish stuff and it's now more in line to what it was before, though the types are merged, moved upwards and easier to deal with.</div>
<div style><br></div><div style>I have reverted all previous tables to the new ones and have deleted the old tables. I also moved the implementation to the cpp files, since no templates were used.</div><div style><br></div>
<div style>Let me know if it's good to commit.</div><div style><br></div><div style>cheers,</div><div style>--renato</div><div style><br></div><div style><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On 16 January 2013 18:33, Renato Golin Linaro <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.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="im">On 16 January 2013 18:22, Benjamin Kramer <span dir="ltr"><<a href="mailto:benny.kra@gmail.com" target="_blank">benny.kra@gmail.com</a>></span> wrote:<br></div><div class="gmail_extra">
<div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><span style="color:rgb(34,34,34)">thread-safety issues aside (I think most c++ implementation implement some kind of safeguard for static initializers), the std::vectors in this patch add static destructors to the code. We try to avoid those in LLVM, can't you just keep them in static constant arrays and only wrap the search on it with another object?</span></div>

</blockquote><div><br></div></div><div>Hi Ben,</div><div><br></div><div>That's a good point. I tried not to use vectors, but I'd have to use all sorts of tricks to keep variable length arrays without increasing in either template code or run time delays, though I haven't thought about the static destructors.</div>

<div><br></div><div>I'll have another look. ;)</div><div><br></div><div>cheers,</div><div>--renato</div></div></div></div>
</blockquote></div><br></div></div>