[PATCH] [MIScheduler] Slightly better handling of constrainLocalCopy when both source and dest are local

Michael Kuperstein michael.m.kuperstein at intel.com
Sun Jan 18 01:03:26 PST 2015


Thanks a lot, Andrew!

As I wrote on the PR, I don't fully understand the surrounding code.
One of the constraints is that the "global" LI has at least one segment after the one that contains the start of the "local" LI. So a "global" LI with exactly one segment will never satisfy it, right? That's the case that I was trying to kill.

Anyway, if you're sure that treating the dest as the "global" works in all cases, I'm perfectly ok with that as well. :-)

Michael


http://reviews.llvm.org/D6823

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list