sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.

Thanks @kmclaughlin , LGTM.



================
Comment at: llvm/include/llvm/IR/IntrinsicsAArch64.td:1115
   class AdvSIMD_SVE_LOGB_Intrinsic  : AdvSIMD_SVE_CNT_Intrinsic;
+  class SVE2_CADD_Intrinsic : AdvSIMD_2VectorArgIndexed_Intrinsic;
+  class SVE2_CMLA_Intrinsic : AdvSIMD_3VectorArgIndexed_Intrinsic;
----------------
nit: The naming is inconsistent not using the `AdvSIMD_` prefix here, although 
we haven't been very consistent with that in this file to begin with.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73636/new/

https://reviews.llvm.org/D73636



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

Reply via email to