[PATCH] D130096: [Clang][AMDGPU] Emit AMDGPU library control constants in clang

[Yaxun Liu via Phabricator via cfe-commits]

yaxunl added inline comments.


================
Comment at: clang/lib/CodeGen/TargetInfo.cpp:9480
+  AddGlobal("__oclc_ISA_version", Minor + Major * 1000, 32);
+  AddGlobal("__oclc_ABI_version", 400, 32);
+}
----------------
jhuber6 wrote:
> yaxunl wrote:
> > should be determined by the code object version option.
> Yes I wasn't sure about this one. Could you elaborate where we derive that?
> Yes I wasn't sure about this one. Could you elaborate where we derive that?

		
CGM.getTarget().getTargetOpts().CodeObjectVersion


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130096