[compiler-rt] r239020 - [ASan] Make binary name reader cross-platform.

Yury Gribov y.gribov at samsung.com
Thu Jun 25 05:21:05 PDT 2015


> I suspect this revision had caused
> https://code.google.com/p/chromium/issues/detail?id=502974
> I haven't found out yet what exactly is causing the problem,
> but I don't think it was a great idea to remove the caching logic from ReadBinaryName
> without replacing calls to it with GetBinaryName
> (if I'm understanding correctly the intention behind GetBinaryName)

Hm, yeah, does not look good. The problem is that I wrote original patch in 2014 and it only got approved in June. During that time folks have been adding new calls to ReadBinaryName which I failed to track as I prepared new versions of patch.

-Y





More information about the llvm-commits mailing list