[libc-commits] [PATCH] D152630: [libc] Enable custom logging in LibcTest
Guillaume Chatelet via Phabricator via libc-commits
libc-commits at lists.llvm.org
Tue Jun 13 06:47:54 PDT 2023
gchatelet added inline comments.
================
Comment at: libc/test/UnitTest/LibcTest.h:53
+template <typename T> struct Matcher : public MatcherBase {
+ bool match(T &t);
+};
----------------
lntue wrote:
> Do you know why the Matcher's interface cannot be `const T&`? Ideally it should not modify the inputs, right?
I traced it down to https://reviews.llvm.org/D75487
I don't see a rationale for this choice so I assume it was just overlooked.
I refrained from fixing it in this patch to prevent noise but since you're mentioning it, here it is.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152630/new/
https://reviews.llvm.org/D152630
More information about the libc-commits
mailing list