[PATCH] D41695: [Metadata] Extend 'count' field of DISubrange to take a metadata node

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 3 16:35:45 PST 2018

probinson added a comment.

In https://reviews.llvm.org/D41695#967020, @rnk wrote:

> I'm concerned this will blow up memory usage of commonly encountered array types.

Which "this"?  Sander's proposal looks like one node per variable-length array, which is surely not very much.  I think Adrian is suggesting always having the count be an expression, which is one node per array type (with some deduplication possible, I hope, for arrays of the same length).  Then I came along and suggested another one for the lower bound, but those would almost always be expressions for 0 and 1, so that's two nodes total per CU.  Unless I'm misunderstanding how DIExpression works (very possible).


More information about the llvm-commits mailing list