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

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 12:13:48 PST 2015


hfinkel added a comment.

In http://reviews.llvm.org/D8705#302648, @atrick wrote:

> I think we need to be conservative in the event that mem_operands are missing, since we haven't made that guarantee--and that's a separate discussion. For example, non-aliasing operations might exist prior to register allocation to handle calling convention (I'm not sure if this is purely theoretical without running tests on all targets).


I agree. We need to make conservative assumptions when mem_operands is missing.


http://reviews.llvm.org/D8705





More information about the llvm-commits mailing list