This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Rename DwarfAccelTable and move header. NFC
ClosedPublic

Authored by JDevlieghere on Jan 25 2018, 4:33 AM.

Details

Summary

This patch renames DwarfAccelTable.{h,cpp} to AccelTable.{h,cpp} and
moves the header to the include dir so it is accessible by the
dsymutil implementation.

Diff Detail

Repository
rL LLVM

Event Timeline

JDevlieghere created this revision.Jan 25 2018, 4:33 AM
aprantl added inline comments.Jan 25 2018, 2:26 PM
include/llvm/CodeGen/AccelTable.h
33 ↗(On Diff #131418)

This should be a file-wide Doxygen comment.

aprantl accepted this revision.Jan 25 2018, 2:29 PM
aprantl added inline comments.
include/llvm/CodeGen/AccelTable.h
35 ↗(On Diff #131418)

Replace this sentence with: The Apple accelerator tables are a precursor of the newer DWARF v5 accelerator tables. Both formats share common design ideas.

This revision is now accepted and ready to land.Jan 25 2018, 2:29 PM

More rewording.

aprantl added inline comments.Jan 25 2018, 2:32 PM
include/llvm/CodeGen/AccelTable.h
69 ↗(On Diff #131418)

DWARF v5

This revision was automatically updated to reflect the committed changes.
llvm/trunk/lib/CodeGen/AsmPrinter/CMakeLists.txt