[PATCH] [PPC] Modify HTM and crypto intrinsics to inherit from GCCBuiltin

Bill Schmidt wschmidt at linux.vnet.ibm.com
Fri Mar 27 15:00:38 PDT 2015


LGTM, with one nit.


================
Comment at: include/llvm/IR/IntrinsicsPowerPC.td:572
@@ -570,2 +571,3 @@
 def int_ppc_altivec_crypto_vshasigmaw :
+            GCCBuiltin<"__builtin_altivec_crypto_vshasigmaw">,                            
             Intrinsic<[llvm_v4i32_ty], [llvm_v4i32_ty,
----------------
The weird spacing and lack of indent here makes me think you've got a whitespace issue.

http://reviews.llvm.org/D8673

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






More information about the llvm-commits mailing list