[llvm-branch-commits] [llvm] RuntimeLibcalls: Add definitions for vector math functions (PR #167026)

[Paul Walker via llvm-branch-commits]

================
@@ -182,10 +182,63 @@ foreach FPTy = ["F32", "F64", "F80", "F128", "PPCF128"] in {
   def MODF_#FPTy : RuntimeLibcall;
 }
 
-foreach VecTy = ["V4F32", "V2F64", "NXV4F32", "NXV2F64"] in {
-  def MODF_#VecTy : RuntimeLibcall;
-  def SINCOS_#VecTy : RuntimeLibcall;
-  def SINCOSPI_#VecTy : RuntimeLibcall;
+defvar F32VectorSuffixes = ["V2F32", "V4F32", "V8F32", "V16F32", "NXV4F32"];
+defvar F64VectorSuffixes = ["V2F64", "V4F64", "V8F64", "NXV2F64"];
----------------
paulwalker-arm wrote:

Sure, but properties like whether a vector function is masked are now in active use, and once this PR lands `hasVectorMaskArgument` becomes out of sync/broken.  Manually updating/verifying that function is fine when there's only a few entries and you just want to get the structure down. However, while the properties are separated like this I don't want to have to do it for all functions.

https://github.com/llvm/llvm-project/pull/167026