grokos added inline comments.
================
Comment at: openmp/libomptarget/include/Ident.h:48-51
+ auto removePath = [](const std::string &path) {
+ std::size_t pos = path.rfind('/');
+ return path.substr(pos + 1);
+ };
----------------
jhuber6 wrote:
> This will probably break with a Windows file path, but I don't think you can
> even build most offloading if you're on Windows. Should I just add a
> processor check?
>
> ```
> #ifdef _WIN32
> #define PATH_DELIMITER '\\'
> #else
> #define PATH_DELIMITER '/'
> #endif
> ```
You are right, libomptarget does not run on Windows, but some fork which stays
in sync with upstream libomptarget may do so and this change may break it. Can
you add the proposed pre-processor check?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87946/new/
https://reviews.llvm.org/D87946
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
