https://bugs.kde.org/show_bug.cgi?id=413547

--- Comment #9 from ahashmi <assad.has...@linaro.org> ---
> Most of the code in this patch is, I assume, boilerplate test-support
> functions taken from other tests in none/tests/arm64.
Yes but it includes additional code for testing half-precision floating-point
instructions. See comment at Float16 definition.

> Should this be present?  It certainly looks like it shouldn't be; if it
> accepts the instruction then it gives no implementation for it, which is
> definitely bogus.
Added implementation and tests for scalar version of half-precision FADDP. The
hope is that this can be the template for future patches supporting v8.2
instructions.

> "most" is somewhat misleading; "half" might be more accurate.  Ditto in 
> randFloat.
Done.

> Just as a general sanity check .. is the way that the capabilities are
> established (by looking at auxv HWCAPS) consistent with the outcome of the
> discussion in bug 414268 ?
Yes.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to