[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 wrote:
> > 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.
> Hmm - please can you check an EXPENSIVE_CHECKS build? I'm seeing > 6min for this.
oh right, I think there is some very expensive checks somewhere in SelectionDAG and there will be a very large number of nodes.... Is there a way to disable tests for builds with expensive checks? Otherwise we probably should remove the test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84328