[PATCH] D41697: [DebugInfo][Metadata] Add support for a DIExpression as 'count' field of DISubrange.

[Paul Robinson via Phabricator via llvm-commits]

probinson added subscribers: rnk, probinson.
probinson added a comment.

In https://reviews.llvm.org/D41697#967493, @aprantl wrote:

> My motivation for disallowing constants after introducing DIExpressions is that I want to avoid having more than one way to represent the same thing, since it complicates the code and brings potential for bugs.


As @rnk points out in https://reviews.llvm.org/D41695, replacing immediate constants with metadata has a real memory cost, and that might be a good reason to preserve allowing both ways.


https://reviews.llvm.org/D41697