This is an archive of the discontinued LLVM Phabricator instance.

[diagtool] Install diagtool
ClosedPublic

Authored by JDevlieghere on May 10 2018, 8:14 AM.

Details

Summary

Although not very well known, diagtool is an incredibly convenient utility for dealing with diagnostics. I believe it's worth adding this to the install target.

Diff Detail

Repository
rL LLVM

Event Timeline

JDevlieghere created this revision.May 10 2018, 8:14 AM

This SGTM, but I wouldn't mind hearing from others. I wonder if this is worth a quick RFC on cfe-dev?

This SGTM, but I wouldn't mind hearing from others. I wonder if this is worth a quick RFC on cfe-dev?

Sounds good, I've created a thread here: http://lists.llvm.org/pipermail/cfe-dev/2018-May/057946.html

vsk added a subscriber: vsk.May 14 2018, 11:19 AM

Sgtm. I think it would be worthwhile to release-note this.

This revision was not accepted when it landed; it landed in state Needs Review.May 16 2018, 3:27 AM
This revision was automatically updated to reflect the committed changes.