This is an archive of the discontinued LLVM Phabricator instance.

[modules] Fix missing includes/typo in LLDB's includes. [NFC]
ClosedPublic

Authored by teemperor on Jan 20 2018, 2:25 PM.

Details

Summary

This patch adds missing includes to the LLDB headers inside include/ as a first step of building LLDB's source with C++ modules. It also fixes this single stds:: typo.

Some quick map why some non-obvious includes were necessary:

  • lldb/lldb-defines.h for LLDB_INVALID_ADDRESS
  • lldb/lldb-types.h for addr_t
  • lldb/lldb-defines.h for DISALLOW_COPY_AND_ASSIG
  • lldb/DataFormatters/TypeSynthetic.h for SyntheticChildrenFrontEnd

Diff Detail

Repository
rL LLVM

Event Timeline

teemperor created this revision.Jan 20 2018, 2:25 PM
teemperor updated this revision to Diff 130784.Jan 20 2018, 3:06 PM
teemperor edited the summary of this revision. (Show Details)
  • Ditched the commented includes.
aprantl accepted this revision.Jan 20 2018, 8:05 PM

Thanks! This looks generally good/useful as a cleanup, too.

include/lldb/Core/ThreadSafeDenseSet.h
49 ↗(On Diff #130784)

Out of curiosity: Why/how did this work before??

This revision is now accepted and ready to land.Jan 20 2018, 8:05 PM
zturner added inline comments.
include/lldb/Core/ThreadSafeDenseSet.h
49 ↗(On Diff #130784)

My guess is nobody is actually using this class so the template was never instantiated.

This revision was automatically updated to reflect the committed changes.