[PATCH] D84328: [ScheduleDAGRRList] Limit number of candidates to explore.

[Florian Hahn via Phabricator via llvm-commits]

fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/test/CodeGen/X86/stress-scheduledagrrlist.ll:12
+  ret void
+}
----------------
RKSimon wrote:
> @fhahn Should this be in x86 or generic codegen tests? I wouldn't have noticed it but its causing a notable increase in runtime for the x86 codegen tests.....
It's intentional for X86 to check that we don't have excessive compile-times (the patch improves compile-time by ~ 100x). On my system, it takes ~1-2 seconds, but if that's too long we can reduce the width of the type further.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84328/new/

https://reviews.llvm.org/D84328