[libcxx-commits] [PATCH] D140278: [libc++] Add __pointer_int_pair

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Dec 20 08:41:44 PST 2022


Mordante added a comment.

Mainly curious, can you give an indication how much space this would save?



================
Comment at: libcxx/include/__math_h/log2i.h:9
+
+#ifndef _LIBCPP___MATH_LOG2I_H
+#define _LIBCPP___MATH_LOG2I_H
----------------
ldionne wrote:
> Why is this not just under `__math`?
Actually I think this file should be removed. Instead we should make `__bit_log2` in `<bit>` available in all language versions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140278



More information about the libcxx-commits mailing list