<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 24, 2013 at 5:36 PM, Thompson, John <span dir="ltr"><<a href="mailto:John_Thompson@playstation.sony.com" target="_blank">John_Thompson@playstation.sony.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Sean,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Why a map instead of just a vector?</span></p></div></div></blockquote><div><br></div><div style>Because the key operation (the "inner loop") is looking up macro expansions (or preprocessor conditionals, or whatever) by their physical source location and seeing if this expansion differs from any previously seen expansions. The lookup by physical source location needs to be fast, hence a map.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u><u></u></span></p>

<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Why do you think it should be set up to do stuff in parallel with threads?  I’ve run the existing version over a fairly large group of headers and it just took
 a few seconds.</span></p></div></div></blockquote><div><br></div><div style>I never said it should be. All I said is that it was easy to see how to parallelize it. However, most codebase-at-a-time tools will probably want to take advantage of multicore someday, so it's a good thing to keep in mind.</div>
<div style><br></div><div style>-- Sean Silva</div></div></div></div>