[clang] [CIR][AArch64] Lower vfma lane builtins (PR #188190)

[Andrzej Warzyński via cfe-commits]

================
@@ -139,11 +139,10 @@ static cir::VectorType getNeonType(CIRGenFunction *cgf, NeonTypeFlags typeFlags,
       cgf->getCIRGenModule().errorNYI(loc, std::string("NEON type: BFloat16"));
     [[fallthrough]];
   case NeonTypeFlags::Float16:
-    if (hasLegalHalfType)
+    if (!hasLegalHalfType)
       cgf->getCIRGenModule().errorNYI(loc, std::string("NEON type: Float16"));
----------------
banach-space wrote:

Why not support this case? Note that `hasLegalHalfType` has recently been updated. Here's the latest definition: https://github.com/llvm/llvm-project/blob/main/clang/lib/CIR/CodeGen/CIRGenBuiltinAArch64.cpp#L203

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