[libc-commits] [PATCH] D159158: [libc] Add a JSON based config option system.

Guillaume Chatelet via Phabricator via libc-commits libc-commits at lists.llvm.org
Fri Sep 1 01:16:25 PDT 2023


gchatelet added inline comments.


================
Comment at: libc/cmake/modules/LibcConfig.cmake:122
+      if(${optname_exists} GREATER -1)
+        # This option is not be overriden so just skip further processing.
+        continue()
----------------
sivachandra wrote:
> gchatelet wrote:
> > ?
> Didn't understand :)
Sorry I should have been more explicit.

I can't parse `This option is not be overriden`, is this `This option is not *to* be *overridden*`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159158



More information about the libc-commits mailing list