[PATCH] D118804: Revert "[Clang] Propagate guaranteed alignment for malloc and others"

2022-02-02 Thread Mauricio Collares via Phabricator via cfe-commits
collares added a comment. It is worth noting that GCC started assuming 16-byte alignment for small objects in 2016, before N2293 was written and accepted into C2x; see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90569#c8 and https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90569#c9 for more recent

[PATCH] D100879: [Clang] Propagate guaranteed alignment for malloc and others

2021-11-20 Thread Mauricio Collares via Phabricator via cfe-commits
collares added a comment. Reproducing oxalica's comment at https://github.com/NixOS/nixpkgs/pull/146817, to make sure it is not missed: "The divergence here is about how to understand the sentence from C17, 7.22.3. There is already an proposal rephrasing it and making it clear that 8-byte allo

[PATCH] D100879: [Clang] Propagate guaranteed alignment for malloc and others

2021-11-20 Thread Mauricio Collares via Phabricator via cfe-commits
collares added a comment. In https://bugzilla.mozilla.org/show_bug.cgi?id=1741454, oxalica discovered this breaks jemalloc-using code, since jemalloc does not guarantee 16-byte alignment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100879/new/