[PATCH] D8705: ScheduleDAGInstrs::buildSchedGraph() handling of memory dependecies rewritten.
Jonas Paulsson via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 29 08:23:37 PST 2016
jonpa added a comment.
OK, submitted as r259201.
Did not see any regressions that called for disabling any tests at least on my machine.
As mentioned before:
- The default value of 1000 for the -dag-maps-huge-region option is very high ("unlimited"). It may very well be that 50 is more reasonable if compile time is precious.
- The other thing that could be done to speed things up is to use a a memory pool for the SUList class - see comment in ScheduleDAGInstrs.h.
More information about the llvm-commits