[PATCH] D26317: Fix use-of-temporary with StringRef in code coverage

2016-11-07 Thread Jordan Rose via cfe-commits
jordan_rose closed this revision. jordan_rose added a comment. Committed as r286122. Repository: rL LLVM https://reviews.llvm.org/D26317 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co

[PATCH] D26317: Fix use-of-temporary with StringRef in code coverage

2016-11-05 Thread Vedant Kumar via cfe-commits
vsk accepted this revision. vsk added a comment. This revision is now accepted and ready to land. Thanks, lgtm. I've made a note to clean this up. Repository: rL LLVM https://reviews.llvm.org/D26317 ___ cfe-commits mailing list cfe-commits@lists.

[PATCH] D26317: Fix use-of-temporary with StringRef in code coverage

2016-11-04 Thread Jordan Rose via cfe-commits
jordan_rose created this revision. jordan_rose added a reviewer: vsk. jordan_rose added a subscriber: cfe-commits. jordan_rose set the repository for this revision to rL LLVM. The fixed code is basically identical to the same loop below, which might indicate an opportunity for refactoring. I just