[clang] [llvm] [AArch64] Remove aarch64_neon_vcvtfp2hf and aarch64_neon_vcvthf2fp (PR #203903)

[Andrzej Warzyński via cfe-commits]

================
@@ -2959,6 +2959,8 @@ CIRGenFunction::emitAArch64BuiltinExpr(unsigned builtinID, const CallExpr *expr,
     return builder.createCast(loc, cir::CastKind::int_to_float, ops[0], ty);
   case NEON::BI__builtin_neon_vcvt_f64_f32:
   case NEON::BI__builtin_neon_vcvt_f32_f64:
+  case NEON::BI__builtin_neon_vcvt_f16_f32:
+  case NEON::BI__builtin_neon_vcvt_f32_f16:
----------------
banach-space wrote:

Could you make sure that these are located in the same relative position as in ARM.cpp, i.e. between:
* `BI__builtin_neon_vcvt_bf16_f32`,
* `BI__builtin_neon_vcvtq_low_bf16_f32`

?

We've been keeping the order consistent between `CIRGenBuiltinAArch64.cpp` and `ARM.cpp` to make the comparison and re-use easier. Thanks!

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