[PATCH] D8705: ScheduleDAGInstrs::buildSchedGraph() handling of memory dependecies rewritten.

Andrew Trick via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 09:37:32 PST 2015


atrick added a comment.

OK. My concern is this:

Two stores both access the same nonaliased location.
One store has it's memory operand stripped for some reason--we don't have any guarantee that memoperands are preserved.
The scheduler now assumes the two stores are independent.


http://reviews.llvm.org/D8705





More information about the llvm-commits mailing list