This is an archive of the discontinued LLVM Phabricator instance.

Cleanup includes: DebugInfo & CodeGen
ClosedPublic

Authored by serge-sans-paille on Mar 9 2022, 1:41 PM.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMar 9 2022, 1:41 PM
serge-sans-paille requested review of this revision.Mar 9 2022, 1:41 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMar 9 2022, 1:41 PM
MaskRay accepted this revision.Mar 11 2022, 5:47 PM
This revision is now accepted and ready to land.Mar 11 2022, 5:47 PM

rebased on main

This revision was automatically updated to reflect the committed changes.

This breaks implicit module on macOS bots: https://green.lab.llvm.org/green/job/lldb-cmake/42098/console

Error message:

/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/CodeGen/MachinePipeliner.h:136:3: error: missing '#include "llvm/ADT/SetVector.h"'; 'SetVector' must be declared before it is used
  SetVector<SUnit *> NodeOrder;
  ^
/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/ADT/SetVector.h:40:7: note: declaration here is not visible
class SetVector {
      ^
/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/lib/CodeGen/AtomicExpandPass.cpp:21:10: fatal error: could not build module 'LLVM_Backend'
#include "llvm/CodeGen/AtomicExpandUtils.h"
 ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.

This can be fixed by adding ADT/SetVector.h to include/llvm/CodeGen/MachinePipeliner.h