dim added a subscriber: bsdjhb.
dim added a comment.

In D150226#4353911 <https://reviews.llvm.org/D150226#4353911>, @aaron.ballman 
wrote:

> One of the major selling points to `constexpr` functions in C++ is that they 
> cannot contain UB -- if your code compiles, it is correct. This bug that 
> we've fixed was another instance of us accidentally allowing UB in a constant 
> expression context when we shouldn't have. FWIW, I pointed out a reasonable 
> workaround for the freebsd issue: https://reviews.llvm.org/D150226#4342516

I tried making that work for gdb, but I failed; there are some complains about 
deleted operator~ that I didn't see before, and I'm going to have to let it 
rest for a bit. I could maybe convince @bsdjhb to submit a solution, since he's 
contributed to gdb and binutils before.


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

Reply via email to