davidxl added a comment. DataValuePtrs won't introduce data races as it is only accessed by the dumping thread. Other than that, I admit I don't like introducing this overhead either. I will prepare a different patch for this. http://reviews.llvm.org/D15258