[PATCH] D101077: [clang][nfc] Split getOrCheckAMDGPUCodeObjectVersion

Jon Chesterfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 22 16:01:22 PDT 2021


JonChesterfield added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:1576
 
-unsigned tools::getOrCheckAMDGPUCodeObjectVersion(
-    const Driver &D, const llvm::opt::ArgList &Args, bool Diagnose) {
+static llvm::opt::Arg *
+getAMDGPUCodeObjectArgument(const Driver &D, const llvm::opt::ArgList &Args) {
----------------
The logic to pick which of multiple arguments wins seems an important thing to keep in one place. I think this is an improvement on the first draft and on D101117


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101077



More information about the cfe-commits mailing list