[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