[Openmp-commits] [PATCH] D95282: [OpenMP] Add source location information to the libomptarget profile

[Joseph Huber via Phabricator via Openmp-commits]

jhuber6 added inline comments.


================
Comment at: openmp/libomptarget/include/SourceInfo.h:77
   std::string removePath(const std::string &path) const {
     std::size_t pos = (OS_WINDOWS) ? path.rfind('\\') : path.rfind('/');
     return path.substr(pos + 1);
----------------
tianshilei1992 wrote:
> This comment might not relate to this patch. I'm just wondering, this header file should be a totally new file. Why the code is not using LLVM style?
Is there anything wrong with it besides the variable names? I just forgot that LLVM uses Pascal case for its variables when I wrote it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95282