[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