[PATCH] D44984: [HIP] Add hip file type and codegen for kernel launching

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 17 12:47:59 PDT 2018


yaxunl marked 2 inline comments as done.
yaxunl added inline comments.


================
Comment at: lib/Frontend/CompilerInvocation.cpp:2109
+      Opts.HIP = true;
+  }
+
----------------
rjmccall wrote:
> yaxunl wrote:
> > rjmccall wrote:
> > > Why is this done here?  We infer the language mode from the input kind somewhere else.
> > It is usually done through CompilerInvocation::setLangDefaults. However, HIP does not have its own input kind nor is it defined as a language standard. Therefore it cannot use CompilerInvocation::setLangDefaults to set Opts.HIP. 
> What are the values of -x if not input kinds or language standards?
I will add hip as input kind and language standard since it really is both.


https://reviews.llvm.org/D44984





More information about the cfe-commits mailing list