[PATCH] D156928: [Clang][AMDGPU] Fix handling of -mcode-object-version=none arg

Fri Aug 4 14:51:14 PDT 2023

yaxunl added a comment.

In D156928#4561890 <https://reviews.llvm.org/D156928#4561890>, @JonChesterfield wrote:

> In D156928#4561849 <https://reviews.llvm.org/D156928#4561849>, @arsenm wrote:
>
>> In D156928#4561811 <https://reviews.llvm.org/D156928#4561811>, @JonChesterfield wrote:
>>
>>> What does code objects version= none mean?
>>
>> Handle any version
>
> So... That should be the default, right? Emit IR that the back end specialises. Or, ideally, the only behaviour as far as the front end is concerned.

Code in the device library depends on a control variable about the code object version. Specifying the code object version in Clang allows internalizing that variable and optimizing code depending on it as early as possible. Not specifying it with Clang will require an LLVM pass in amdgpu backend to define that control variable after linking and it has to have an external linkage. This may lose optimization. Also, you need a way to not specify it in FE but specify it in BE. This just complicates things without much benefits.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156928/new/

https://reviews.llvm.org/D156928