[PATCH] [Sanitizer RT] Put the module name string ownership in Symbolizer in order
Timur Iskhodzhanov
timurrrr at google.com
Tue Mar 31 05:55:09 PDT 2015
I think `ModuleNameOwner` might eventually turn into something bigger (e.g. `ModulesHistory`?) and/or it can be reused in other parts of the RTL (e.g. `StringsCache`/`StringsOwner`?), so I think it makes sense to keep it as a separate self-contained type.
I liked the idea of `CheckLocked` though, so I've decided to just pass a pointer to the `Symbolizer::mu_` mutex to the `ModuleNameOwner` ctor.
Please take a look at r233687.
http://reviews.llvm.org/D8666
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list