GCC maintainers:
Here is my respnses to the review comments by Kewen. Unfortunately,
Kewen is no longer working on GCC power.
I will submit an updated version of the patch with Kewen's suggested
changes.
Carl
On 8/9/24 3:11 AM, Kewen.Lin wrote:
Hi Carl,
on 2024/8/8 01:15, Carl Love wrote:
GCC maintainers:
The following patch removes two redundant built-ins __builtin_vsx_vperm_8hi and
__builtin_vsx_vperm_8hi_uns. The built-ins are covered by the overloaded
vec_perm built-in.
The patch has been tested on Power 10 LE and BE with no regressions.
Please let me know if it is acceptable for mainline. Thanks.
Carl
-------------------------------------------------------------
rs6000, remove built-ins __builtin_vsx_vperm_8hi and __builtin_vsx_vperm_8hi_uns
The two built-ins __builtin_vsx_vperm_8hi and __builtin_vsx_vperm_8hi_uns
are redundant. The are covered by the overloaded vec_perm built-in. The
built-ins are not documented and do not have test cases.
OK for trunk, maybe also mention this is a follow up of r15-1923, thanks!
Yes, added:
The removal of these built-ins was missed in commit gcc r15-1923 on
7/9/2024.
to the patch description.
Carl
