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

Siva Chandra via Phabricator via libc-commits libc-commits at lists.llvm.org
Fri Sep 1 10:25:29 PDT 2023


sivachandra 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()
----------------
gchatelet wrote:
> 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*`?
Oh! Fixed now.


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