This is an archive of the discontinued LLVM Phabricator instance.

Add more DWARF 5 language constants.
ClosedPublic

Authored by brucem on Feb 5 2015, 2:56 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

brucem updated this revision to Diff 19389.Feb 5 2015, 2:56 AM
brucem retitled this revision from to Add more DWARF 5 language constants..
brucem updated this object.
brucem edited the test plan for this revision. (Show Details)
brucem added a reviewer: echristo.
brucem added a subscriber: Unknown Object (MLST).
echristo edited edge metadata.Feb 5 2015, 9:44 AM

Hi Bruce,

The DWARF5 standard hasn't been released yet and so the numbers aren't
technically final yet so I've been avoiding making those changes if
possible. Mind holding off a couple weeks until (I hope) the preliminary
review copy is released?

Thanks!

-eric

echristo accepted this revision.Feb 5 2015, 10:21 AM
echristo edited edge metadata.

After discussion on the lldb bug I've been convinced that we can label these as "subject to change" but allow people to get testing going.

LGTM.

-eric

This revision is now accepted and ready to land.Feb 5 2015, 10:21 AM
brucem updated this revision to Diff 19523.Feb 6 2015, 5:45 PM
brucem edited edge metadata.

Update for recent changes on HEAD.

LGTM still. Thanks!

-eric

This revision was automatically updated to reflect the committed changes.