[PATCH] Use ".arch_extension" ARM directive to support hwdiv on krait

Renato Golin renato.golin at linaro.org
Thu Feb 26 08:09:51 PST 2015


With the comment inline sorted, LGTM, thanks!


================
Comment at: lib/Target/ARM/ARMAsmPrinter.cpp:707
@@ -701,1 +706,3 @@
 
+  // We consider krait as a "cortex-a9" + hwdiv CPU
+  // Enable hwdiv through ".arch_extension idiv"
----------------
Can you move this up with the first check? I think it'll be easier to "remove Krait" later, as the FIXME hints to.

http://reviews.llvm.org/D7819

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list