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

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 4 11:44:06 PST 2018

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.


More information about the llvm-commits mailing list