[PATCH] D142078: [llvm] Move bit counting functions to bit.h (NFC)

Slava Zakharin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 19 21:52:09 PST 2023


vzakhari added inline comments.


================
Comment at: llvm/include/llvm/Support/MathExtras.h:17
 #include "llvm/ADT/bit.h"
-#include "llvm/Support/Compiler.h"
 #include <cassert>
----------------
This removal breaks the build with GCC-9*, because `__has_builtin` seems to be available starting GCC-10.  Now we are missing this useful code from `Compiler.h`:
```
#ifndef __has_builtin
# define __has_builtin(x) 0
#endif
```

Can you please bring the `include` back to both `MathExtras.h` and `bit.h`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142078



More information about the llvm-commits mailing list