fpetrogalli added a comment.

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.


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