[PATCH] D13255: [LAA] LLE 2/6: Fix a NoDep case that should be a Forward dependence

[Renato Golin via llvm-commits]

rengolin added a subscriber: rengolin.
rengolin added a comment.

IIUC, this new dependency analysis won't trigger anything in the compiler to change its behaviour, so even if the analysis is not complete yet (and need an extra dep to trigger the memory check), it still shouldn't break any existing code. Is that correct?

I wonder if this could be squashed into some future commit (3rd~6th) to avoid the weird comments on the test case, even if temporary. But it's not a big deal, since no one will be crazy enough to pick a stable master where the head is "[2/6]" of some patch. :)

As long as it doesn't break any bot (which I doubt it will), this should be fine, too.


http://reviews.llvm.org/D13255