[PATCH] D68300: [HIP] Use option -nogpulib to disable linking device lib

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 3 09:36:56 PDT 2019


tra added inline comments.


================
Comment at: include/clang/Driver/Options.td:606
+def flink_builtin_bitcode : Flag<["-"], "flink-builtin-bitcode">,
+  Flags<[CC1Option]>, HelpText<"Link builtin bitcode for HIP device compilation.">;
+def fno_link_builtin_bitcode : Flag<["-"], "fno-link-builtin-bitcode">;
----------------
hliao wrote:
> tra wrote:
> > yaxunl wrote:
> > > ashi1 wrote:
> > > > Since this is a more generic approach, we won't need to specify HIP ?
> > > this patch only implemented this option for HIP. If it is used for other languages, this help text should be updated.
> > Hmm. Cuda currently uses `-nocudalib` for essentially the same purpose (Sort of like `-nostdlib`, but for CUDA). Perhaps we should consolidate all these into `-nogpulib` and alias `-nocudalib` to it. 
> how about other relevant options, such as replacing cuda-device-only with gpu-device-only or hip-device-only to avoid confusing with CUDA.
We seem to be doing exactly that, only incrementally. In general it does make sense to consolidate the flags where the functionality is common.
`cuda-device-only`, `cuda_host_only`, `cuda_compile_host_device`  and `cuda-gpu-arch` could be generalized to `gpu-<something>`. 

This should be a separate patch, though.


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

https://reviews.llvm.org/D68300





More information about the cfe-commits mailing list