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

Thu Jul 23 11:59:27 PDT 2020

RKSimon added inline comments.

================
Comment at: llvm/test/CodeGen/X86/stress-scheduledagrrlist.ll:12
+  ret void
+}
----------------
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.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84328