asb added a comment.

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

> Could you please add a test? Given that the current version of 
> https://reviews.llvm.org/D44886 enables linker relaxation by default in the 
> backend, shouldn't -mno-relax cause -relax to be set?


Sorry, I misready D444886. Still, we should do Features.push_back("-relax") at 
least in the case where -mno-relax is given explicitly, meaning we don't have 
to rely on the backend default.


Repository:
  rC Clang

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