This is an archive of the discontinued LLVM Phabricator instance.

[XRay][tools] Rename llvm-xray filenames from .cc -> .cpp (NFC)
ClosedPublic

Authored by dberris on Apr 30 2018, 11:39 PM.

Details

Summary

This brings the filenames in accordance to the style guide and LLVM
conventions for C++ filenames.

As suggested by rnk@ in D46068.

Event Timeline

dberris created this revision.Apr 30 2018, 11:39 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 1 2018, 5:48 PM
This revision was automatically updated to reflect the committed changes.
thakis added a subscriber: thakis.May 3 2018, 7:30 AM
thakis added inline comments.
llvm/trunk/tools/llvm-xray/CMakeLists.txt
9 ↗(On Diff #144816)

Out of interest, what is LLVM_XRAY_TOOLS for? It's used only in this file as far as I can tell -- why not list all the sources in the add_llvm_tool call below directly?

dberris added inline comments.May 3 2018, 7:35 AM
llvm/trunk/tools/llvm-xray/CMakeLists.txt
9 ↗(On Diff #144816)

Good question. Was copying what I thought was a convention in other tools.

llvm/tools/llvm-xray/llvm-xray.cc