[PATCH] D24587: Output optimization remarks in YAML

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 09:32:19 PDT 2016


anemet added a comment.

For the record, the committed version adds the new namespace 'ore' as we talked about.

I was also able to simplify the underlying NV machinery.  Essentially, I always create a string key to string value pair for each argument (if the arg is string, the key is just 'String').

I think there is room to optimize this with some move semantics but I leave that to a follow-on patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D24587





More information about the llvm-commits mailing list