[PATCH] D138202: [OHOS] Add support for OpenHarmony

Pavel Kosov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 6 03:40:01 PST 2023


kpdev42 marked an inline comment as done.
kpdev42 added inline comments.


================
Comment at: llvm/include/llvm/ADT/Triple.h:742
+  /// LiteOS default enviroment is also OHOS, but omited on triple.
+  bool isOHOSFamily() const { return isOpenHOS() || isOSLiteOS(); }
+
----------------
jrtc27 wrote:
> arsenm wrote:
> > As someone who's never heard of this operating system, this helper name isn't particularly enlightening
> What if I give aarch64-liteos-gnu?
We currently assume environment for `liteos` is always `ohos`. Thus `gnu` env will be ignored.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138202



More information about the llvm-commits mailing list