[libc-commits] [PATCH] D152503: [libc][NFC] Simplify LibcTest and trim down string allocations

Guillaume Chatelet via Phabricator via libc-commits libc-commits at lists.llvm.org
Fri Jun 9 02:02:51 PDT 2023


gchatelet marked 2 inline comments as done.
gchatelet added inline comments.


================
Comment at: libc/test/UnitTest/FPMatcher.h:24
 
-template <typename T, __llvm_libc::testing::TestCondition Condition>
+using __llvm_libc::testing::TestCond;
+
----------------
sivachandra wrote:
> gchatelet wrote:
> > sivachandra wrote:
> > > Instead of this, I think we can get rid of the `fputil` namespace which is mostly a legacy remnant.
> > > Instead of this, I think we can get rid of the `fputil` namespace which is mostly a legacy remnant.
> > 
> > AFAICT this means removing the `fputil` namespace everywhere correct? If so I will probably do this in a separate patch as it's quite big.
> I meant only in the test related components. You can do it separately.
👍


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152503



More information about the libc-commits mailing list