This is an archive of the discontinued LLVM Phabricator instance.

[libunwind] Clean up macro usage.
ClosedPublic

Authored by rs on Mar 17 2017, 7:11 AM.

Diff Detail

Event Timeline

rs created this revision.Mar 17 2017, 7:11 AM
EricWF accepted this revision.Mar 27 2017, 8:07 PM

LGTM minus the suggested cleanup.

include/__libunwind_config.h
14–15

This should have an empty definition to avoid people depending on the value 1.

20

This should have an empty definition to avoid people depending on the value 1.

src/config.h
34–35

This should have an empty definition to avoid people depending on the value 1.

51–52

This should have an empty definition to avoid people depending on the value 1.

55–56

This should have an empty definition to avoid people depending on the value 1.

63–64

This should have an empty definition to avoid people depending on the value 1.

This revision is now accepted and ready to land.Mar 27 2017, 8:07 PM
rs updated this revision to Diff 93646.Mar 31 2017, 8:37 AM
rs marked 6 inline comments as done.
This revision was automatically updated to reflect the committed changes.