shiva0217 added a comment.

In https://reviews.llvm.org/D44888#1068806, @asb wrote:

> Thanks Kito. -mrelax and -mno-relax currently only affect the backend. For 
> completeness, I think this patch needs to pass the appropriate flag to the 
> linker depending on relax/no-relax.


Hi Alex. RISCVToolChain class does not define yet, so we can't pass --no-relax 
in RISCV::Linker::ConstructJob. Should we passing --no-relax when we introduce 
RISCVToolchain? Or there is another place we could implement passing RISCV 
linker flags?


Repository:
  rL LLVM

https://reviews.llvm.org/D44888



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

Reply via email to