[all-commits] [llvm/llvm-project] 8c6319: [libc++] Add a missing assertion in std::span's co...

Louis Dionne via All-commits all-commits at lists.llvm.org
Thu Aug 11 12:32:03 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8c6319e30a357fb9b25db09b6f5fc9cf3e7c4aab
      https://github.com/llvm/llvm-project/commit/8c6319e30a357fb9b25db09b6f5fc9cf3e7c4aab
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2022-08-11 (Thu, 11 Aug 2022)

  Changed paths:
    M libcxx/include/span
    A libcxx/test/libcxx/containers/views/views.span/span.cons/assert.iter_sent.pass.cpp
    A libcxx/test/libcxx/containers/views/views.span/span.cons/assert.iter_size.pass.cpp
    A libcxx/test/libcxx/containers/views/views.span/span.cons/assert.other_span.pass.cpp
    A libcxx/test/libcxx/containers/views/views.span/span.cons/assert.range.pass.cpp

  Log Message:
  -----------
  [libc++] Add a missing assertion in std::span's constructor

Also, add missing tests for assertions in span constructors. Now I
believe that all of std::span's API should be hardened, and all the
assertions should have a corresponding test.

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




More information about the All-commits mailing list