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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
