[PATCH] D81359: [ELF] Add --[no-]relax for RISC-V

[George Rimar via Phabricator via llvm-commits]

grimar added inline comments.


================
Comment at: lld/ELF/Config.h:188
   bool printIcfSections;
+  bool relax;
   bool relocatable;
----------------
You do not need this.


================
Comment at: lld/ELF/Driver.cpp:977
       args.getLastArgValue(OPT_print_symbol_order);
+  config->relax = args.hasFlag(OPT_relax, OPT_no_relax, true);
   config->rpath = getRpath(args);
----------------
And this.


================
Comment at: lld/ELF/Options.td:340
 
+defm relax: BB<"relax",
+  "Enable target-specific relaxations (default)",
----------------
And this too I think.


================
Comment at: lld/ELF/Options.td:631
 
 // Options listed below are silently ignored for now for compatibility.
 def: F<"detect-odr-violations">;
----------------
You just need to list the option(s) here I believe.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81359/new/

https://reviews.llvm.org/D81359