[PATCH] D17738: [NVPTX] Fix function identifiers that are invalid in PTX and a bug fix for the case of name collisions.

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 30 17:05:22 PST 2017


rnk added a comment.

In https://reviews.llvm.org/D17738#661115, @hfinkel wrote:

> One option is that we add a function to LLVM get an available separator character, which can default to '.', but we set to '$' for nvptx, and use that for generating new names at the IR level. Thoughts?


This seems practical. Perhaps it could be part of the name mangling scheme already encoded in DataLayout?


https://reviews.llvm.org/D17738





More information about the llvm-commits mailing list