[PATCH] Add support for annotated minimum dependence distance
hfinkel at anl.gov
hfinkel at anl.gov
Tue Mar 10 16:02:39 PDT 2015
In http://reviews.llvm.org/D5123#137651, @ArchDRobison wrote:
> I believe that support for forward lexical dependences was the item that required partial ordering support, but was orthogonal to this patch, which addresses minimum dependence distance.
I was not under the impression that it is orthogonal (or it is a prerequisite). Right now, we only have a way to specify that a loop is completely data parallel (no loop carried dependencies at all), so it does not really matter. It does matter, however, if we have a minimum dependency distance, because that distance can have a validity dependence on lexical dependencies (forward and backward) that are sensitive to potential reordering.
Also, as a separate matter, the unroller, when it does partial unrolling, needs to update this metadata (to divide the distances by the unrolling factor).
http://reviews.llvm.org/D5123
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list