add bfloat header test

added testing

replanted arg passing test from other patch

addressed review comments and some related general changes:

• renamed IsSoftFloatABI -> IsFloatABISoftFP
• split bfloat tests out of arm-mangle-16bit-float.cpp
• bfloat shouldn't be checking for NativeHalfType
• err_cast_(from|to)_bfloat -> err_cast_(from|to)_bfloat16
• reworded comments and other minor fixes

moving 'CartesianProductWith' to more apt patch

no explicit float-abi cmdline arg should default to softfp

making testing more robust as per review comments

Thanks Francesco, that was enlightening, and great info to know. I took on all your suggestions, and changed the testing accordingly.

addressed review comments, most of all changed license header on the generated bfloat file

In D79710#2041418, @LukeGeeson wrote:

Can you update the commit message in this differential as well please? Same for the other commits :)

added testcases, following fp16's lead

Hiya Francesco, thanks for the thorough comment :)

Note that these argument passing tests discussed above aren't present in the follow-up patch at the moment. I'm currently implementing them.

Having a think, I think the calling convention test would be more meaningful if we have a bit more bfloat lowering support. Which we have in the bfloat lowering follow-up patch: https://reviews.llvm.org/D79712

adhere to attribution conventions: change author to Ties, add all the contributors

adhere to patch attribution conventions: change author to Ties, add all the contributors

adhere to attribution conventions

adhere to patch attribution conventions

added bfloat SVE MVT type

fix bug that failed webassembly test

redo: addressing review comments and adding Simon Tatham to contributers

addressing review comments and adding Simon Tatham to contributers

@LukeGeeson: you're already mentioned :) See the commits tab on this review. But Simon Tatham needs a mention as well. I shall add him.

@fpetrogalli: Yes, good point. It was on my todo. I had tested these changes with such a snippet, so there's no good reason I didn't add it in the first place.

fix bug in ReplaceBITCASTResults

changes:

• addressed asmith's comments
• Bfloat doesn't depend on half cmdline options and correct code is generation for AArch32 hard and softfp CC

In D79712#2033801, @john.brawn wrote:

Could you add some tests (unless they're added in a later patch)?

@asmith: Thanks for reviewing :) Fixed your suggestions in the coming revision.

Hi there, a gentle ping: does this look good to you?

So are we happy with the patch as it is? Anybody else wants to LGTM?

In D78190#2020701, @stuij wrote:

In D78190#2018685, @fpetrogalli wrote:

3. Do you need those changes in the Hexagon and x86 backend? Could they be submitted separately, with some testing?

This is a backend agnostic patch. If the Hexagon and/or x86 communities want to make use of the IR type in some way, then yes, they can for sure submit the necessary patches.

In D78190#2018685, @fpetrogalli wrote:

1. Shouldn't we test also that the parser is happy with the following expressions?

added vector tests

Hi all. Are you OK for me to commit this change?