pcwang-thead abandoned this revision.
pcwang-thead added a comment.

In D144853#4157629 <https://reviews.llvm.org/D144853#4157629>, @MaskRay wrote:

> I object to this change which further complicates the build system and makes 
> the difference difficult to observe by users.
> If your `clang` executable is at `bin/clang`, just create 
> `bin/riscv64-unknown-linux-gnu.cfg` with `-mcpu=xxx`.
> You may configure `LLVM_DEFAULT_TARGET_TRIPLE=riscv64-unknown-linux-gnu` so 
> that the driver doesn't need extra `--target=`.

Thanks! I almost forget about config file.
I think this change is unnecessary now and thanks all for your time to review 
this patch. :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144853

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

Reply via email to