This is an archive of the discontinued LLVM Phabricator instance.

[LIBC} Put todo on making cpp::array::end() method constexpr
Needs ReviewPublic

Authored by goldstein.w.n on Apr 13 2023, 8:05 PM.

This revision needs review, but there are no reviewers specified.

Summary

michaelrj, sivachandra

Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptApr 13 2023, 8:05 PM

goldstein.w.n requested review of this revision.Apr 13 2023, 8:05 PM

Path

Size

libc/

src/

__support/

CPP/

3 lines

Show All 37 Lines	template <class T, size_t N> struct array {
constexpr size_t size() const { return N; }		constexpr size_t size() const { return N; }

constexpr bool empty() const { return N == 0; }		constexpr bool empty() const { return N == 0; }

constexpr iterator begin() { return Data; }		constexpr iterator begin() { return Data; }
constexpr const_iterator begin() const { return Data; }		constexpr const_iterator begin() const { return Data; }

constexpr iterator end() { return Data + N; }		constexpr iterator end() { return Data + N; }

		// TODO: Marking this constexpr causes a variety of test/build failures. Look
		// into that and fix.
const_iterator end() const { return Data + N; }		const_iterator end() const { return Data + N; }
};		};

} // namespace cpp		} // namespace cpp
} // namespace __llvm_libc		} // namespace __llvm_libc

#endif // LLVM_LIBC_SRC_SUPPORT_CPP_ARRAY_H		#endif // LLVM_LIBC_SRC_SUPPORT_CPP_ARRAY_H