[PATCH] D37179: [ARM] Tidy-up condition-code support functions

[Javed Absar via Phabricator via llvm-commits]

javed.absar added inline comments.


================
Comment at: lib/Target/ARM/Utils/ARMBaseInfo.h:51
+  switch (CC) {
+  default: llvm_unreachable("Unknown condition code");
+  case EQ: return NE;
----------------
fhahn wrote:
> While you are at it, would it make sense to remove the default case there and add the llvm_unreachable after the switch like in ARMCondCodeToString?
> 
> This would give the compiler a better chance to warn about missing patterns, but that's only a minor point, as it's unlikely that CondCodes will change.
Thanks. Will do so.


https://reviews.llvm.org/D37179