[PATCH] D35548: [mips] Teach the driver to accept -m(no-)gpopt.
    Simon Atanasyan via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Tue Jul 18 05:33:37 PDT 2017
    
    
  
atanasyan added inline comments.
================
Comment at: lib/Driver/ToolChains/Clang.cpp:1490
+      GPOpt->claim();
+  }
+
----------------
Could it be rewritten a bit shorter?
```
bool NoAbiCalls =
    ABICalls && ABICalls->getOption().matches(options::OPT_mno_abicalls);
bool WantGPOpt = GPOpt && GPOpt->getOption().matches(options::OPT_mgpopt);
if (NoAbiCalls && (!GPOpt || WantGPOpt)) {
  CmdArgs.push_back("-mllvm");
  CmdArgs.push_back("-mgpopt=1");
} else {
  CmdArgs.push_back("-mllvm");
  CmdArgs.push_back("-mgpopt=0");
}
if (GPOpt)
  GPOpt->claim();
```
https://reviews.llvm.org/D35548
    
    
More information about the cfe-commits
mailing list