[PATCH] D20729: [mips] Compact branch policy setting
Simon Atanasyan via cfe-commits
cfe-commits at lists.llvm.org
Fri May 27 06:39:43 PDT 2016
atanasyan accepted this revision.
atanasyan added a comment.
This revision is now accepted and ready to land.
LGTM with some nits
================
Comment at: lib/Driver/Tools.cpp:1446
@@ +1445,3 @@
+ CmdArgs.push_back(Args.MakeArgString("-mips-compact-branches=" + Val));
+
+ } else
----------------
Redundant empty line.
================
Comment at: lib/Driver/Tools.cpp:7084
@@ +7083,3 @@
+ // mips32r6 and mips64r6 have compact branches.
+
+ return (bool)llvm::StringSwitch<int>(CPU)
----------------
Redundant empty line.
================
Comment at: lib/Driver/Tools.cpp:7085
@@ +7084,3 @@
+
+ return (bool)llvm::StringSwitch<int>(CPU)
+ .Case("mips32r6", true)
----------------
Is it possible to re-write this line as:
```
return llvm::StringSwitch<bool>(CPU)
```
http://reviews.llvm.org/D20729
More information about the cfe-commits
mailing list