Skip to content

Merge clang r322246 into 6.0 branch: [X86] Fix vpshrd builtins to require an ICE for their constant argument to match vpshld. #35263

@topperc

Description

@topperc
Bugzilla Link 35915
Resolution FIXED
Resolved on Jan 17, 2018 08:27
Version 6.0
OS Windows NT
Blocks #35152
CC @zmodem,@RKSimon

Extended Description

This commit makes the frontend generate an error for this code because the last argument should be an integer constant expression

__m512i test_mm512_mask_shrdi_epi64(__m512i __S, __mmask8 __U, __m512i __A, __m512i __B, unsigned x) {
return _mm512_mask_shrdi_epi64(__S, __U, __A, __B, x);
}

Without this check in the frontend it goes to the backend and throws a fatal error in the backend during isel. The frontend error is a better user experience.

The _mm512_mask_shrdi_epi64 was introduced in late December 2017 so doesn't exist in any releases before 6.0

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions