[PATCH] D21843: [Driver][OpenMP] Create tool chains for OpenMP offloading kind.

[Samuel Antao via cfe-commits]

sfantao added a comment.

Hi Hal,

Thanks for the review!



================
Comment at: include/clang/Basic/DiagnosticDriverKinds.td:163
+def err_drv_expecting_fopenmp_with_fopenmp_targets : Error<
+  "The option -fopenmp-targets must be used in conjunction with a -fopenmp option compatible with offloading.">;
+def warn_drv_omp_offload_target_duplicate : Warning<
----------------
hfinkel wrote:
> This message does not tell the user how they might make their -fopenmp option "compatible with offloading." Please make sure the message does, or is has an associated hint message which does.
> 
Ok, the message is now: `The option -fopenmp-targets must be used in conjunction with a -fopenmp option compatible with offloading, please use -fopenmp=libomp or -fopenmp=libiomp5.`


https://reviews.llvm.org/D21843