On Wed, Feb 18, 2015 at 10:56 AM, Benjamin Kramer <[email protected]> wrote: > On Wed, Feb 18, 2015 at 7:45 PM, Benjamin Kramer > <[email protected]> wrote: >> Author: d0k >> Date: Wed Feb 18 12:45:54 2015 >> New Revision: 229719 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=229719&view=rev >> Log: >> Driver: Fix use of dangling std::string temporary >> >> What's going on here is that the ternary operator produces a std::string >> rvalue >> that the StringRef points to. I'd hoped bugs like this were a thing of the >> past >> with our asan testing but apparently this code path is only used when LLVM is >> configured with a custom --with-c-include-dirs setting. >> >> Unbreaks bootstrapping with GCC5 on Fedora (PR22625), patch by Jonathan >> Wakely! > > Hans, > > this should be safe for 3.6, can you squeeze it in?
Merged in r229746. Thanks, Hans _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
