[PATCH] D52776: [OptRemarks] Add library for parsing optimization remarks

[Francis Visoiu Mistrih via Phabricator via llvm-commits]

thegameg updated this revision to Diff 168106.
thegameg marked 10 inline comments as done.
thegameg added a comment.

Thanks Jessica for the review.

- Addressed comments.
- Removed skipping logic.
- Use `llvm::Error`. Much much better! Thanks!
- Update tests.


https://reviews.llvm.org/D52776

Files:
  include/llvm-c/OptRemarks.h
  lib/CMakeLists.txt
  lib/LLVMBuild.txt
  lib/OptRemarks/CMakeLists.txt
  lib/OptRemarks/LLVMBuild.txt
  lib/OptRemarks/OptRemarksParser.cpp
  tools/llvm-opt-report/CMakeLists.txt
  tools/llvm-opt-report/OptReport.cpp
  unittests/CMakeLists.txt
  unittests/OptRemarks/CMakeLists.txt
  unittests/OptRemarks/OptRemarksParsingTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52776.168106.patch
Type: text/x-patch
Size: 41684 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181003/03f77479/attachment-0001.bin>