This is an archive of the discontinued LLVM Phabricator instance.

[llvm-dwarfdump][Stats] Add the License header
ClosedPublic

Authored by djtodoro on Feb 7 2020, 3:12 AM.

Diff Detail

Event Timeline

djtodoro created this revision.Feb 7 2020, 3:12 AM
Herald added a project: Restricted Project. · View Herald Transcript
This revision is now accepted and ready to land.Feb 7 2020, 3:17 AM

By the way, this is completely independent of anything else in the patch series, so please commit it as soon as you can.

Yes, I do not know why I put in the stack. I will land this. Thanks!

This revision was automatically updated to reflect the committed changes.
aprantl added inline comments.Feb 7 2020, 8:49 AM
llvm/tools/llvm-dwarfdump/Statistics.cpp
1

This is doubly broken :-)

  1. The language mark is spelled -*- C++ -*-
  2. It only makes sense to put this into a .h file where the programming language is ambiguous. A .cpp file is always C++.

Oh, I'm sorry, I didn't know :(

djtodoro updated this revision to Diff 243186.Feb 7 2020, 9:05 AM

@aprantl Is it better?

djtodoro reopened this revision.Feb 7 2020, 9:05 AM
This revision is now accepted and ready to land.Feb 7 2020, 9:05 AM

Yes, thanks!

This revision was automatically updated to reflect the committed changes.