[Openmp-commits] [libcxx] [llvm] [flang] [openmp] [mlir] [clang] [clang-tools-extra] [clang-tidy] Replace memcpy with std::copy (PR #74663)

Piotr Zegar via Openmp-commits openmp-commits at lists.llvm.org
Sat Dec 9 01:28:22 PST 2023


https://github.com/PiotrZSL requested changes to this pull request.

**Missing tests.**
Lot of issues.
Personally i would consider modernize-use-std-copy-fill as an name, and cover also memset, as this would be the same code. And I would convert this into std::copy_n or std::full_n.

The only thing when std::copy should eb used would be if call to memset would be:
memset(other_begin, begin, end-begin);

https://github.com/llvm/llvm-project/pull/74663


More information about the Openmp-commits mailing list