This is an archive of the discontinued LLVM Phabricator instance.

[llvm-{debuginfod,ml,objdump,symbolizer}, dsymutil] Enable multicall driver
ClosedPublic

Authored by avillega on Aug 10 2023, 6:34 PM.

Diff Detail

Event Timeline

avillega created this revision.Aug 10 2023, 6:34 PM
Herald added a reviewer: MaskRay. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
avillega requested review of this revision.Aug 10 2023, 6:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 10 2023, 6:34 PM
MaskRay accepted this revision.Aug 13 2023, 9:29 PM
This revision is now accepted and ready to land.Aug 13 2023, 9:29 PM
phosek accepted this revision.Aug 14 2023, 11:55 PM

LGTM

This revision was landed with ongoing or failed builds.Aug 16 2023, 6:10 PM
This revision was automatically updated to reflect the committed changes.

This change broke the multicall driver build:

/b/s/w/ir/x/w/llvm-llvm-project/llvm/tools/llvm-ml/llvm-ml.cpp:58:10: fatal error: 'Opts.inc' file not found
   58 | #include "Opts.inc"
      |          ^~~~~~~~~~

This change broke the multicall driver build:

/b/s/w/ir/x/w/llvm-llvm-project/llvm/tools/llvm-ml/llvm-ml.cpp:58:10: fatal error: 'Opts.inc' file not found
   58 | #include "Opts.inc"
      |          ^~~~~~~~~~

Sorry for the breakage. Seems like the Opts.inc file was generated in a previous build on my machine and cached and that's why it didn't reproduce. Doing a clean build showed the problem immediately.

avillega reopened this revision.Aug 17 2023, 2:50 PM
This revision is now accepted and ready to land.Aug 17 2023, 2:50 PM
avillega updated this revision to Diff 551283.Aug 17 2023, 2:50 PM

Reland:
Explicitly add the missing llvm-ml missing dependency

This revision was landed with ongoing or failed builds.Aug 17 2023, 4:27 PM
This revision was automatically updated to reflect the committed changes.