[llvm] ConstraintSystem: replace data structure with Matrix (PR #98895)

Ramkumar Ramachandra via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 16 07:02:29 PDT 2024


artagnon wrote:

> I think one large upfront allocation wouldn't hurt as much as repeatedly calling malloc/free to resize the underlying storage. I think things can be improved by doing a dry-run, upper-bounding the number of rows and columns, and performing the entire allocation upfront.

Another thing I noticed was a call to a copy-constructor that copies the underlying storage. Is it really necessary to copy the underlying storage?

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


More information about the llvm-commits mailing list