[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