[libcxx-commits] [PATCH] D85095: Fix libcxx build on riscv32

Luís Marques via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Sep 17 08:44:55 PDT 2020


luismarques added a comment.

I suppose that an alternative to this implementation might be to tweak the place of use of `__NR_futex`, like is done in [1]. Arguably that's cleaner. Or, in the place where you're currently checking for riscv32, check for the case where `__NR_futex_` isn't defined but `__NR_futex_time64` is, and #define the former in terms of the latter.

[1]: https://sourceware.org/legacy-ml/libc-alpha/2020-01/msg00686.html


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

https://reviews.llvm.org/D85095



More information about the libcxx-commits mailing list