asb added a comment.
I wonder if it would be safer to change this patch so it adds -mrelax and
-mno-relax but doesn't compile with linker relaxation by default. That makes it
easier to test linker relaxation support, and gives more time for testing
before then flipping to -mrelax as the default.
Comment at: test/Driver/riscv-features.c:8
+// RUN: %clang -target riscv32-unknown-elf -### %s -mno-relax 2>&1 | FileCheck
+// RELAX: "-target-feature" "+relax"
We need a another RUN line and CHECK here to determine the default whether
+relax is passed or not when the user specifies neither -mrelax or -mno-relax.
cfe-commits mailing list