[PATCH] D41296: Limit size of non-GlobalValue name

serge guelton via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 28 00:18:45 PST 2017


On Wed, Dec 27, 2017 at 09:58:42PM +0000, David Blaikie wrote:
> Is silently truncating the right thing here, or should this be an error?
> (otherwise I'm assuming you could have two different names that, when
> truncated, become one name & produce possibly unintended semantics?)

That's ok, setName already handles conflicting names by appending a
numeral suffix.


More information about the llvm-commits mailing list