[llvm] r176214 - The FastISEL should be fast. But when we record statistics we use atomic operations to increment the counters.

[Nadav Rotem]

Hi Eric, 

> This is what I was talking about that Nadav never bothered to respond to.

You wrote me less than 24 hours ago, and I did not have time to write back. 

> Whether or not it made sense to put all of the statistics as only firing during
> debug builds and then making the change in the statistics framework rather
> than #if NDEBUG for each set of statistics and the assorted increments.
> 

I don't have any preference. Do you prefer one way or the other ?

I kept NumDAGBlocks and NumFastIselBlocks, because they are called once per blocks and not once per instructions. I don't mind if we put them behind #ifndef NDEBUG as well.  

Thanks,
Nadav
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130228/17c44684/attachment.html>