[PATCH] D46423: [WIP, RISCV] Support .option relax and .option norelax
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 24 06:37:07 PDT 2018
asb added a comment.
I'd still err towards `.option norelax` meaning "do what I say". If a user freely intermixes .option relax and .option norelax within a section then there's a whole bunch of things that might break. Simply ignoring the norelax doesn't seem right, and picking out particular cases (e.g. symbol diffs) where we'll act as if relaxation was enabled also seems a bit dodgy. I still think that erroring might be safest and simplest path.
Repository:
rL LLVM
https://reviews.llvm.org/D46423
More information about the llvm-commits
mailing list