[all-commits] [llvm/llvm-project] 4163f6: [libc++] [ranges] Fix a missing auto(x) cast in ra...

[Quuxplusone via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4163f61f29f47d3e0d82cec2ba4cf8997d4b6820
      https://github.com/llvm/llvm-project/commit/4163f61f29f47d3e0d82cec2ba4cf8997d4b6820
  Author: Arthur O'Dwyer <arthur.j.odwyer at gmail.com>
  Date:   2022-01-12 (Wed, 12 Jan 2022)

  Changed paths:
    M libcxx/include/__ranges/data.h
    M libcxx/test/std/ranges/range.access/data.pass.cpp
    M libcxx/test/std/ranges/range.req/range.refinements/contiguous_range.compile.pass.cpp

  Log Message:
  -----------
  [libc++] [ranges] Fix a missing auto(x) cast in ranges::data.

Also remove some bogus `std::forward`s. My impression is that these
forwards were actually harmless, because `ranges::begin(FWD(t))` is
always identical to `ranges::begin(t)` (except when it's ill-formed,
and that can't happen in this case). However, they're also superfluous
and don't reflect the wording in the standard, so let's eliminate them.

Differential Revision: https://reviews.llvm.org/D117043


  Commit: 9be193bc58b356e2d2e0bddff59a404358e2c75e
      https://github.com/llvm/llvm-project/commit/9be193bc58b356e2d2e0bddff59a404358e2c75e
  Author: Arthur O'Dwyer <arthur.j.odwyer at gmail.com>
  Date:   2022-01-12 (Wed, 12 Jan 2022)

  Changed paths:
    M libcxx/include/__ranges/data.h
    M libcxx/test/std/ranges/range.access/begin.pass.cpp
    M libcxx/test/std/ranges/range.access/data.pass.cpp
    M libcxx/test/std/ranges/range.access/empty.pass.cpp
    M libcxx/test/std/ranges/range.access/end.pass.cpp
    M libcxx/test/std/ranges/range.access/size.pass.cpp
    M libcxx/test/std/ranges/range.access/ssize.pass.cpp

  Log Message:
  -----------
  [libc++] [ranges] Finish ADL-proofing ranges::data.

This should have been part of D116239.


Compare: https://github.com/llvm/llvm-project/compare/b9499e14d24f...9be193bc58b3