aaron.ballman added a comment. In D150226#4340166 <https://reviews.llvm.org/D150226#4340166>, @manojgupta wrote:
> https://github.com/bminor/binutils-gdb/blob/master/include/diagnostics.h > > gdb only suppresses the warning. So this patch will likely break gdb. > > As per commit: > https://github.com/bminor/binutils-gdb/commit/ae61525fcf456ab395d55c45492a106d1275873a > > Since the current code does what we want, and I don't see any way of doing > it > differently, ignore -Wenum-constexpr-conversion around it. Hmmm, I'm not certain, but I *think* this does the right thing: `std::is_signed_v<decltype(1 ? std::declval<Ty>() : std::declval<std::underlying_type_t<Ty>>())>;` https://godbolt.org/z/MWY4e1e9e CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150226/new/ https://reviews.llvm.org/D150226 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits