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

Sumanth Gundapaneni sgundapa at codeaurora.org
Tue Feb 24 18:42:59 PST 2015


================
Comment at: lib/Target/ARM/ARMAsmPrinter.cpp:709
@@ +708,3 @@
+  if (STI.isKrait() && (STI.hasDivide() || STI.hasDivideInARMMode()))
+    ATS.emitArchExtension("idiv");
+
----------------
rengolin wrote:
> I was looking around and emitting plain text of a hardcoded value here would be a bad idea.
> 
> I think you need to create/append an enum and do like FPU/CPU/ARCH.
Renato, Can you be more specifc on this ?
If you can point me to an existing implementation, it would be nice.

http://reviews.llvm.org/D7819

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






More information about the llvm-commits mailing list