[PATCH] D27751: Use after move bug fixes

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 08:34:34 PST 2016


mehdi_amini added inline comments.


================
Comment at: tools/llvm-profdata/llvm-profdata.cpp:53
   if (E.isA<InstrProfError>()) {
     handleAllErrors(std::move(E), [&](const InstrProfError &IPE) {
       instrprof_error instrError = IPE.get();
----------------
I'm wondering why there is a check for error before calling handleAllErrors here? (@lhames while you're at it...)


https://reviews.llvm.org/D27751





More information about the llvm-commits mailing list