This is an archive of the discontinued LLVM Phabricator instance.

[LLVM] [CodeView] Fix some Clang-tidy modernize-use-default, modernize-use-override and Include What You Use warnings; other minor fixes
AbandonedPublic

Authored by Eugene.Zelenko on Nov 16 2016, 7:13 PM.

Details

Summary

I checked this patch on my own build on RHEL 6. Regressions were OK.

Diff Detail

Repository
rL LLVM

Event Timeline

Eugene.Zelenko retitled this revision from to [LLVM] [CodeView] Fix some Clang-tidy modernize-use-default, modernize-use-override and Include What You Use warnings; other minor fixes.
Eugene.Zelenko updated this object.
Eugene.Zelenko set the repository for this revision to rL LLVM.
Eugene.Zelenko added a subscriber: llvm-commits.
mehdi_amini edited edge metadata.Nov 16 2016, 7:41 PM

+1 for post-commit

Eugene.Zelenko abandoned this revision.Nov 17 2016, 10:40 AM

Per Zachary and Mehdi suggestion to use post-commit reviews.