[PATCH] D142218: [MC] Store implicit ops immediately after the TargetInsts table. NFC.

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 27 06:30:05 PDT 2023


foad added a comment.

In D142218#4223825 <https://reviews.llvm.org/D142218#4223825>, @foad wrote:

> In D142218#4223664 <https://reviews.llvm.org/D142218#4223664>, @RKSimon wrote:
>
>> @foad MSVC builds seem to be breaking due to this:
>> https://lab.llvm.org/buildbot/#/builders/86/builds/53509
>>
>>   LLVMARMAsmParser.lib(ARMAsmParser.cpp.obj) : error LNK2001: unresolved external symbol "class llvm::MCInstrDesc const * const llvm::ARMDescs" (?ARMDescs at llvm@@3QBVMCInstrDesc at 1@B)
>>   bin\llc.exe : fatal error LNK1120: 1 unresolved externals
>>   ninja: build stopped: subcommand failed.
>
> Thanks - I just noticed that too. I have a quick fix but can't push it now due to some partial github outage.

Pushed ffab44bd960c6392a0ed1f945cc145a2a2643c8c <https://reviews.llvm.org/rGffab44bd960c6392a0ed1f945cc145a2a2643c8c>


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142218/new/

https://reviews.llvm.org/D142218



More information about the llvm-commits mailing list