On Fri, Jun 04, 2021 at 01:03:58AM +0000, Liu, Hongtao wrote:
> Thanks for the review.
> Yes, you're right, AVX512VL parts are already guaranteed by
> ix86_hard_regno_mode_ok.
>
> Here is updated patch.
One remaining thing, could you try to modify the testcase back to
#include <x86intrin.h> and using intrinsics instead of the target builtins,
so that next time we replace some builtins we don't have to adjust the
testcase (and of course verify that without your patch it still ICEs and
with your patch it doesn't)?
Ok for trunk with that change.
Jakub
