[libcxx-commits] [PATCH] D133124: [libc++][lit][AIX] Enable test case last_write_time.pass.cpp for AIX

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Sep 9 06:39:09 PDT 2022


ldionne requested changes to this revision.
ldionne added inline comments.
This revision now requires changes to proceed.


================
Comment at: libcxx/test/std/input.output/filesystems/fs.op.funcs/fs.op.last_write_time/last_write_time.pass.cpp:504-505
+        {"dir, past_time", dir, past_time}
+#if !defined(_AIX)
+        ,
         {"file, before_epoch_time", file, before_epoch_time},
----------------
Instead, can we move the comment above to here, always define `before_epoch_time` & friends, and only have one `#ifdef` block? That will reduce the scope of the workaround.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133124



More information about the libcxx-commits mailing list