This is an archive of the discontinued LLVM Phabricator instance.

Import/update constants from the DWARF 5 public review draft document.
ClosedPublic

Authored by aprantl on Oct 27 2016, 5:13 PM.

Details

Summary

I noticed two incompatible changes:

  1. DW_AT_dwo_id was removed because the dwo_id was moved into the CU header. Note that the attribute value was reserved, so LLVM can continue to emit it without causing ambiguity for now.
  2. The one change that could cause a problem here is that DW_AT_ranges_base has been replaced by DW_AT_rnglist_base.

David/Eric: Would merely renaming the constant break any existing DWARF (4) tooling?

Diff Detail

Event Timeline

aprantl updated this revision to Diff 76138.Oct 27 2016, 5:13 PM
aprantl retitled this revision from to Import/update constants from the DWARF 5 public review draft document..
aprantl updated this object.
aprantl added reviewers: echristo, dblaikie, probinson.
aprantl added subscribers: clayborg, llvm-commits.
aprantl updated this object.Oct 27 2016, 5:14 PM
aprantl accepted this revision.Oct 28 2016, 8:41 AM
aprantl added a reviewer: aprantl.

Signed off by echristo via mail.

This revision is now accepted and ready to land.Oct 28 2016, 8:41 AM
aprantl closed this revision.Oct 28 2016, 4:31 PM

285421, 285444, 285448.