[libc-commits] [PATCH] D151231: [libc][NFC] Move the Linux file implementation to a subdirectory

[Tue Ly via Phabricator via libc-commits]

lntue added inline comments.


================
Comment at: libc/src/__support/File/CMakeLists.txt:37
+add_subdirectory(${LIBC_TARGET_OS})
+set(target_file libc.src.__support.File.${LIBC_TARGET_OS}.${LIBC_TARGET_OS}_file)
+set(target_dir libc.src.__support.File.${LIBC_TARGET_OS}.${LIBC_TARGET_OS}_dir)
----------------
jhuber6 wrote:
> lntue wrote:
> > Since the targets are moved to separate sub-folders, shall we drop the prefix `${LIBC_TARGET_OS}_` from the target name?
> We could, I was mostly just copying the interface from how we handle OSUtil.
Can you change it so that it matches other folders' structure?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151231