[libc-commits] [PATCH] D101991: [libc] Allow target architecture customization

[Siva Chandra via Phabricator via libc-commits]

sivachandra accepted this revision.
sivachandra added inline comments.
This revision is now accepted and ready to land.


================
Comment at: libc/cmake/modules/LLVMLibCObjectRules.cmake:44
   )
-  if(ADD_OBJECT_COMPILE_OPTIONS)
-    target_compile_options(
-      ${fq_target_name}
-      PRIVATE ${ADD_OBJECT_COMPILE_OPTIONS}
-    )
-  endif()
+  _set_common_compile_options(${ADD_OBJECT_COMPILE_OPTIONS})
+  target_compile_options(${fq_target_name} PRIVATE ${LIBC_COMMON_COMPILE_OPTIONS})
----------------
Instead of `_set_*`, I think a function with name `_get_*` will be more appropriate here. The function should also take an additional arg for the result:

```
_get_all_compile_options(all_opts ...)
target_compile_options(${fq_target_name} PRIVATE ${all_opts})
```

This probably a nitty comment but motivated by the fact that `common` is not suitable in `add_object_library`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101991