This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] use build_vector_from_val in more places
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Nathan Froyd <froydnj at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 1 Nov 2010 07:39:34 -0700
- Subject: Re: [PATCH] use build_vector_from_val in more places
- References: <20101030214503.GE6758@nightcrawler>
On Sat, Oct 30, 2010 at 2:45 PM, Nathan Froyd <froydnj@codesourcery.com> wrote:
> The recently-introduced build_vector_from_val function can be used in
> several places, centralizing TREE_LIST use and making it easier to
> remove at a later point. ?The assertion change in build_vector_from_val
> is needed so as to not ICE while building libgfortran and is, I think,
> more correct in any event.
>
> Tested on x86_64-unknown-linux-gnu. ?OK to commit?
>
> -Nathan
>
> ? ? ? ?* fold-const.c (build_zero_vector): Use build_vector_from_val.
> ? ? ? ?* tree-vect-loop.c (get_initial_def_for_induction): Likewise.
> ? ? ? ?(get_initial_def_for_reduction): Likewise.
> ? ? ? ?* tree-vect-slp.c (vect_get_constant_vectors): Likewise.
> ? ? ? ?* tree-vect-stmts.c (vect_get_vec_def_for_operand): Likewise.
> ? ? ? ?* tree.c (build_one_cst): Likewise.
> ? ? ? ?(build_vector_from_val): Use types_compatible_p for assert.
>
This caused:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46259
H.J.