This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH GCC][01/13]Introduce internal function IFN_LOOP_DIST_ALIAS
- From: Richard Sandiford <richard dot sandiford at linaro dot org>
- To: "Bin.Cheng" <amker dot cheng at gmail dot com>
- Cc: "gcc-patches\@gcc.gnu.org" <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 26 Jun 2017 11:23:30 +0100
- Subject: Re: [PATCH GCC][01/13]Introduce internal function IFN_LOOP_DIST_ALIAS
- Authentication-results: sourceware.org; auth=none
- References: <VI1PR0802MB21760955A0BC2ABC60FC62D4E7CD0@VI1PR0802MB2176.eurprd08.prod.outlook.com> <CAHFci28Xy6zKU8Uv28QE1hz6-Rp6=z37F+rGpZViemr5KELBaw@mail.gmail.com>
Just a couple of cosmetic things:
"Bin.Cheng" <amker.cheng@gmail.com> writes:
> @@ -225,6 +225,15 @@ struct GTY ((chain_next ("%h.next"))) loop {
> builtins. */
> tree simduid;
>
> + /* For loops generated by distribution with runtime alias checks, this
> + is a unique identifier of the original distributed loop. Generally
> + it is the number of the original loop. IFN_LOOP_DIST_ALIAS builtin
> + uses this id as its first argument. Give a loop with an id, we can
> + look upward in dominance tree for the corresponding IFN_LOOP_DIST_ALIAS
> + buildin. Note this id has no meanling after IFN_LOOP_DIST_ALIAS is
s/meanling/meaning/
> +/* Fold LOOP_DIST_ALIAS internal call stmt according to KEEP_P and update
> + any immediate uses of it's LHS. Stmt is folded to its second argument
s/it's/its/
Thanks,
Richard