[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