• Used existing masked.load intrinsic in .ll test that checks the verifier catches bad type when passing a fixed-width vector where a scalable is expected.
• Removed definition of abs and neg intrinsics that were previously used in the above test, these are implemented in the follow-up patch.
• Updated call to CreateIntrinsic in IRBuilderTest to make types argument empty for call to fp-convert intrinsic, only required for overloaded types.

In D65931#1635304, @RKSimon wrote:

Is it worth adding in ISD::ABS support here ?

@greened sorry for the delayed response, thanks for taking a look at this.

• Moved the intrinsic implementation for abs/neg (without codegen) from D65931 so this patch can be tested.
• Added tests for IRBuilder to test scalable vectors work as expected when using intrinsics through the IRBuilder interface. This required implementing an intrinsic that is explicitly defined with scalable vectors, e.g. LLVMType<nxv4i32>, an SVE floating-point convert intrinsic was used for this. Also tested the behaviour of the overloaded type LLVMScalarOrSameVectorWidth with scalable vectors using the existing masked load intrinsic.
• Replace uses of getNumElements with getElementCount when handling intrinsic type LLVMScalarOrSameVectorWidth so it can correctly support scalable vectors.
• Added an .ll test to test the Verifier catches a bad intrinsic argument when passing a fixed-width predicate (mask) to the abs intrinsic where a scalable is expected.

In D65930#1620641, @sdesmalen wrote:

Specifically to assert that passing a fixed-width vector where a scalable vector is expected (and vice-versa), fails.

I've posted a follow-up patch (D65931) to Phabricator that tests this functionality by implementing intrinsics for a couple of SVE arithmetic instructions.

@ostannard Thanks for review!

• Improved tests
• Fixed up comments

In D65385#1604643, @rengolin wrote:

Right, ArchExtKind was supposed to be shared between Arm and AArch64, but that ship has sailed a long time ago. LGTM, thanks!

This was split up into smaller patches, all have now been merged. Abandoning this revision.

Addressed Momchil's comments: explicitly enable/disable arch features in negative directive tests.