tbaederr wrote: > This will need a release note. I'm curious why the removed code was there in > the first place. Any idea @tbaederr ?
No idea, it's been there since the original commit introducing `__builtin_bit_cast`. Maybe just an optimization, or maybe something else in that loop used to assert that the class isn't empty. Aside from the release not, this LGTM. https://github.com/llvm/llvm-project/pull/82383 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits