[PATCH] Value profiling - patchset 2 - merge intended

Justin Bogner mail at justinbogner.com
Fri Jun 19 18:52:35 PDT 2015


Justin Bogner <mail at justinbogner.com> writes:
> I've gone ahead and committed a clang-formatted version of this in
> r240206.

I had to revert this, as some bots were failing:

  http://lab.llvm.org:8011/builders/sanitizer-windows/builds/5640
  http://lab.llvm.org:8011/builders/lldb-x86-win7-msvc/builds/5757
  http://lab.llvm.org:8011/builders/lldb-x86-windows-msvc/builds/7067

On looking at the error it looks like you're move'ing from a vector that
you've taken by reference. That seems pretty wrong. I guess you wanted
an rvalue reference there?

Anyway, please look into the errors and send a patch that fixes the
issues.



More information about the llvm-commits mailing list