[PATCH] D142403: [build] Force building LLVMTargetParser before other non-LLVM components.

Tom Stellard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 23 13:54:21 PST 2023


tstellar added a comment.

In D142403#4074905 <https://reviews.llvm.org/D142403#4074905>, @fpetrogalli wrote:

> In D142403#4074885 <https://reviews.llvm.org/D142403#4074885>, @tstellar wrote:
>
>> This is still the wrong change IMO.  I don't know, maybe I'm not being clear, but I don't 'think you ever actually tried my suggestion in D141581 <https://reviews.llvm.org/D141581> which was to leave  the RISCVTargetParserTableGen Depends as is and add the pseudo targets in llvm/cmake/modules/LLVMConfig.cmake.in.
>
> We'll get there! I think I missed the "leave the `RISCVTargetParserTableGen` in the `DEPENDS`, sorry about that.

No problem!  Thanks for sticking with it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142403



More information about the cfe-commits mailing list