[llvm] ConstraintElim: add dry-run routine to fail early (PR #99670)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 22 14:12:23 PDT 2024


fhahn wrote:

> > Managed to eek out another small optimization: http://llvm-compile-time-tracker.com/compare.php?from=c5ee3c05ca61f3fae11337c5853aee7b450a9dc6&to=f0d674c9f5aa601c0e5472f208ff96f3d855d9b6&stat=instructions%3Au.
> > @dtcxzyw Could you kindly re-check against the latest commit?
> 
> I also wonder if this suffices to establish whether the early conservative estimation of rows/columns outweighs the somewhat noticeable code duplication, maybe worth it to run SPEC with this before further reviewing?

I can check that this is NFC from a code-gen perspective for SPEC and some other larger workloads in the next few days. Please ping me if you don't hear from me by Thursday :) 

https://github.com/llvm/llvm-project/pull/99670


More information about the llvm-commits mailing list