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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to