PATCH: More thorough cleanup of NVPTX-generated global names

Jingyue Wu jingyue at google.com
Thu Mar 27 10:07:51 PDT 2014


LGTM


On Wed, Mar 26, 2014 at 4:24 PM, Eli Bendersky <eliben at google.com> wrote:

> Hello,
>
> As discussed in PR19099, the solution in r203483 was imperfect. While we
> could fix global names when they are emitted in directives, this does not
> work for their usage within instructions. Instruction emission is
> hard-coded through MI and NVPTX does not control this.
>
> A more thorough solution is a target-specific IR pass that replaces all
> invalid characters in symbol names before they go into codegen. This makes
> sure that invalid characters will never appear in symbols in PTX.
>
> An added benefit is that this avoids collisions because Value->setName
> does automatic collision resolution.
>
> Once this goes in, most of r203483 can be reverted (I'll take care of
> that).
>
> PTAL,
> Eli
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140327/8cb61a60/attachment.html>


More information about the llvm-commits mailing list