Cory Fields
Cory Fields
Fixed up now. Ironically this doesn't actually eliminate the warning as I'd hoped. I will leave this open in case anyone finds it interesting, but no need to think of...
Needs a quick linter fix.
Hmm, just throwing this out there, maybe more trouble than it's worth... Rather than hacking the boost m4, most of which we don't really require anymore now that we don't...
Concept ACK. I'm having trouble finding where ppc64 became unsupported in guix though. There were lots of refactors and I'm guessing some wildcard change is maybe responsible? I'd like to...
FWIW, `c++filt` output: ```bash $ echo "_ZNSt6vectorI7uint256SaIS0_EE17_M_realloc_insertIJEEEvN9__gnu_cxx17__normal_iteratorIPS0_S2_EEDpOT_" | c++filt ``` `void std::vector::_M_realloc_insert(__gnu_cxx::__normal_iterator) ` I _believe_ [this is the function in question](https://github.com/gcc-mirror/gcc/blob/fab08d12b40ad637c5a4ce8e026fb43cd3f0fad1/libstdc%2B%2B-v3/include/bits/vector.tcc#L453). Though it's not clear to me if the function...
Huh, yeah, I'm not sure either. I never bumped into this with my testing. Will have a look and try to PR a fix.
It took me a while to reproduce... indeed clang-15 does not complain, but clang-17 does. Seeing as it detected something that clang-15 missed, but smarter tracking _could potentially_ understand vars...
FWIW @hebasto's solution looks correct (and necessary) to me. An alternative would be to use c++20's `std::assume_aligned` if we could guarantee the input's alignment, but looking at the usage that's...
It'd be worth taking upstream as well.
Very nice! Strong concept ACK, but obviously we can't use this until all features we need are supported. But it's great to see that linking finally works! For a little...