[PATCH] D95007: [CUDA][HIP] Add -fuse-cuid

Jon Chesterfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 19 17:04:34 PST 2021


JonChesterfield added inline comments.


================
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:2307
+  // Only alphanumeric and underscore is allowed in -cuid option.
+  if (auto *A = Args.getLastArg(OPT_cuid_EQ)) {
+    const char *V = A->getValue();
----------------
Why this limitation? In particular I can imagine people using an id based on a filesystem location, which introduces /\:. and possibly other characters.


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

https://reviews.llvm.org/D95007



More information about the cfe-commits mailing list