asb added a comment. In https://reviews.llvm.org/D44888#1111920, @shiva0217 wrote:
> In https://reviews.llvm.org/D44888#1109361, @asb wrote: > > > This is looking good to me, just needs an update to address this > > <https://reviews.llvm.org/D44888#inline-411895> request for a test in > > riscv-features.c that demonstrates the default +relax/-relax setting. > > > Hi Alex. I added the testing line on https://reviews.llvm.org/D47127 which is > the patch we turn on relaxation as default. Do you think it's ok? I saw that. I don't mean to be too nit-picky, but there's an advantage in also testing the default in this patch - it tests the current behaviour and makes it obvious in the follow-up patch that the behaviour actually changed. 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
