echristo added inline comments.

================
Comment at: lib/Basic/Targets.cpp:5709
@@ -5715,1 +5708,3 @@
+
+    return false;
   }
----------------
compnerd wrote:
> Please collapse this:
> 
>     return Name == "generic" || llvm::AArch64::parseCPUArch(Name) != 
> llvm::ARM::AK_INVALID;
... why doesn't this actually set the cpu?


Repository:
  rL LLVM

http://reviews.llvm.org/D21277



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to