[PATCH] [Polly][Refactor] Simplify (Vector)BlockGenerator

Tobias Grosser tobias at grosser.es
Wed Oct 8 03:55:47 PDT 2014


On 08/10/2014 05:17, Johannes Doerfert wrote:
> Hi grosser, sebpop, dpeixott, simbuerg,
>
>    This removes an unnecessary cast and a unnecessary local variable.

LGTM.

Your [Refactor] flags implies this sometimes, but it is probably clearer
to write 'No functional change' for changes that are not intended to 
change behaviour. This seems to be what people commonly use use.

Tobias




More information about the llvm-commits mailing list