[PATCH] D39005: [OpenMP] Clean up variable and function names for NVPTX backend

[Gheorghe-Teodor Bercea via Phabricator via cfe-commits]

gtbercea added a comment.

In https://reviews.llvm.org/D39005#900226, @jlebar wrote:

> > I'd be interested to get the ball rolling in regard to coming up with a fix for this. I see some suggestions in past patches. Some help/clarification would be much appreciated.
>
> Happy to help, but I'm not sure what to offer beyond the link in Art's previous comment.


Thanks Justin. Perhaps if we could start by clarifying this statement "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."  as well as this statement "This seems practical. Perhaps it could be part of the name mangling scheme already encoded in DataLayout?".

The first question that comes to mind is what is the link between data layout and name mangling conventions?


Repository:
  rL LLVM

https://reviews.llvm.org/D39005