[polly] r270416 - Optimistic assume required invariant loads to be invariant
Tobias Grosser via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 7 10:16:54 PDT 2016
On 05/23/2016 12:40 PM, Johannes Doerfert via llvm-commits wrote:
> Author: jdoerfert
> Date: Mon May 23 05:40:54 2016
> New Revision: 270416
>
> URL: http://llvm.org/viewvc/llvm-project?rev=270416&view=rev
> Log:
> Optimistic assume required invariant loads to be invariant
>
> Before this patch we bailed if a required invariant load was potentially
> overwritten. However, now we will optimistically assume it is actually
> invariant and, to this end, restrict the valid parameter space as well as the
> execution context with regards to potential overwrites of the location.
Perfect. I really like this commit message. This is the kind of summary
that I missed in some of my earlier reviews.
Best,
Tobias
More information about the llvm-commits
mailing list