On Thu Feb 6 15:03:08 2025 +0000, William Horvath wrote:
#if !defined(__clang__) && (defined(__GNUC__) && (__GNUC__ < 8 || (__GNUC__ == 8 && __GNUC_MINOR__ < 1)) && (defined(__x86_64__) || defined(__i386__)))
since this was [fixed](https://gcc.gnu.org/git/?p=gcc.git&a=commit;h=d8c40eff56f69877b33c697ded...) quite a while ago? This would make the include guards that your patch adds elsewhere quite messy, but I think the intrinsics should be preferred if they're not broken. Is there another reason we avoid them completely, besides what's in the GCC bugzilla link?
This change is out of scope. Please open a new MR instead.