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

James Y Knight via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 2 09:42:51 PST 2022


jyknight added a comment.

In D118804#3290803 <https://reviews.llvm.org/D118804#3290803>, @xbolva00 wrote:

> GCC also does same assumptions

Looking at GCC: GCC only assumes 4-byte alignment on i386, and 8-byte alignment on x86-64, which is why it hasn't actively broken users, unlike the clang change.

However, I think that's also still a bug in GCC. It should not be assuming 8-byte alignment for a 1-byte allocation.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118804/new/

https://reviews.llvm.org/D118804



More information about the cfe-commits mailing list