[PATCH] D142022: [Clang][OpenMP] Fix handling of -mcode-object-version for OpenMP

Saiyedul Islam via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 18 09:39:24 PST 2023


saiislam marked an inline comment as done.
saiislam added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:7116
   if (Triple.isAMDGPU()) {
-    handleAMDGPUCodeObjectVersionOptions(D, Args, CmdArgs);
+    handleAMDGPUCodeObjectVersionOptions(D, C.getArgs(), CmdArgs);
 
----------------
yaxunl wrote:
> why do you need to change Args to C.getArgs() ?
You are right. Don't need it. Removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142022



More information about the cfe-commits mailing list