[PATCH] DataFlowSanitizer: Prefix the name of each instrumented function with "dfs$".

[Evgeniy Stepanov]

================
Comment at: lib/Transforms/Instrumentation/DataFlowSanitizer.cpp:360
@@ +359,3 @@
+  std::string SearchStr = ".symver " + GVName + ",";
+  size_t Pos = Asm.find(SearchStr);
+  if (Pos != std::string::npos) {
----------------
Peter Collingbourne wrote:
> Evgeniy Stepanov wrote:
> > This looks really scary. I'm not sure such things belong to the platform-independent instrumentation pass.
> Yes this is horrible.  Unfortunately libstdc++ uses .symver directives and without this compilation of libstdc++ will fail.  I'm not sure where else to put it.  (In principle we could send the module inline asm through an MC streamer which does any symbol renaming, but that's probably overkill for this one use case.)
Could there be more than 1 .symver directive for the same globalvalue?


http://llvm-reviews.chandlerc.com/D1373