================
@@ -64,6 +64,21 @@ struct ARMVectorIntrinsicInfo {
   }
 };
 
+struct ARMScalableVectorIntrinsicInfo {
----------------
banach-space wrote:

Please document `ARMScalableVectorIntrinsicInfo` and `ARMVectorIntrinsicInfo`. 
In particular, please make sure that the rationale for not re-using one 
`struct` is clear. There's too much code in this area with unclear rationale.

Also, `ARMScalableVectorIntrinsicInfo` -> 
`AArch64SVEAndSMEVectorIntrinsicInfo`? And `ARMVectorIntrinsicInfo` -> 
`ARMNeonVectorIntrinsicInfo`? That would be more self-documenting to me.

https://github.com/llvm/llvm-project/pull/202618
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to