
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>Hi</div><div><br></div><div>As requested, I have ran the address sanitizer tests. I ran them many times, and could not find any reproducible failures.</div><div>I did notice some intermittent failures that came and went depending on what revision I had gotten from SVN.</div><div><br></div><div>Due to concern for use-after-free I have included a patch to clang that allows strings passed to ParseCommandLineOptions() from the CreateTargetMachine() function to persist (this was the only location in LLVM or clang where I found ParseCommandLineOptions being called with automatic memory); what it does is strdups the args for -debug-pass, -limit-float-precision as well as backend args and stores the result to a global vector of StringRefs. This vector is only populated for unique strings to avoid leaking repeatedly.</div><div><br></div><div>The other three patches include changes to the CommandLine library to enable StringRef (which I already sent out), changes to places using cl::opt<std::string> to use StringRef (also already sent out), and also the requested changes to the CommandLine documentation.</div><div><br></div><div>I think this should cover all the basis. I’ll check in if this looks ok to you all.</div><div><br></div><div>Thanks</div><div><br></div><div>-Puyan</div><div><br></div><div><br></div><div></div></body></html>