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

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

In D80475#2052869 <https://reviews.llvm.org/D80475#2052869>, @orivej wrote:

> 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.


If string::begin returns a `std::string::iterator` (everywhere else in the world...), names in `std::` are looked up ("The innermost enclosing namespaces in the classes added to the set") when you call an unqualified `transform`.

If string::begin returns `const char *`, ADL does not fire and `std::` is not looked up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80475