[PATCH] D80475: [TargetLoweringObjectFileImpl] Use llvm::transform

[Orivej Desh via Phabricator via llvm-commits]

orivej added a comment.

I suppose that the newer versions are generally more restrictive than the older ones. Old libstdc++ used to bring std names into the global scope.

I have found what causes the build failure on my part: it is that string.begin() and end() return a `char *`, not an iterator class in the std namespace: https://github.com/catboost/catboost/blob/08a65a2e/contrib/libs/cxxsupp/libcxx/include/string#L697

Could you please commit on my behalf?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80475