Re: [tree-ssa-lno] vectorizer patch.

[Dorit Naishlos <DORIT at il dot ibm dot com>]

Hi Richard,

Regarding the new build_vector_type() utility you added in tree.c to
replace the target hook - where did you envision that the check to verify
that the vector type is supported would take place? With your change, it
allows vector types that are not supported to go through. Is it ok if I add
the check (test VECTOR_MODE_SUPPORTED_P) in tree.c:build_vector_type()?

thanks,
dorit



                                                                       
                      Richard Henderson                                
                      <rth@redhat.com>         To:       Daniel Berlin <dberlin@dberlin.org>
                                               cc:       gcc-patches@gcc.gnu.org, Dorit Naishlos/Haifa/IBM@IBMIL, Pop Sébastian
                      02/01/2004 23:40          <pop@gauvain.u-strasbg.fr>
                                               Subject:  Re: [tree-ssa-lno] vectorizer patch.
                                                                       




On Fri, Jan 02, 2004 at 04:34:06PM -0500, Daniel Berlin wrote:
> Well, the vectorizer uses it...

That's not the point.  At no point does the target need to be involved
in the creation or selection of a vector type node.

I've just restructured tree.c slightly to export a build_vector_type.
I'll commit that change shortly.


r~