On 29/09/18 10:56 +0200, Marc Glisse wrote:
Hello,here is a clang-friendly version of the patch (same changelog), tested a while ago. Is it ok or do you prefer something like the+ if(this->_M_impl._M_start._M_offset != 0) __builtin_unreachable(); version suggested by François?
I don't think __builtin_unreachable would improve the clarity of the code. The patch is OK for trunk, thanks.
