On September 25, 2019 5:29:55 PM GMT+02:00, Eric Botcazou <ebotca...@adacore.com> wrote: >> The following patch relaxes the condition under which we force >> VOIDmode by making all non-integral types where the extraction >> size matches the type size (thus isn't "bitfieldish") use the >> mode of the extraction type. > >Wouldn't TREE_CODE (TREE_TYPE (exp)) == VECTOR_TYPE be sufficient? At >least >this would still be in keeping with the comment...
For the PR it would be good enough. Though I wonder what the original reason for the mode handling was. Was it to avoid not naturally aligned modes for strict align targets? Or modes for non-mode size entities? Richard.
