[all-commits] [llvm/llvm-project] 7e830f: [Clang] Partially fix m68k alignments (#144740)

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7e830f76711f0dfc66780ea13cf780b8760b458b
      https://github.com/llvm/llvm-project/commit/7e830f76711f0dfc66780ea13cf780b8760b458b
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2025-07-01 (Tue, 01 Jul 2025)

  Changed paths:
    M clang/lib/Basic/Targets/M68k.cpp
    M clang/test/CodeGen/bitfield-access-pad.c
    M clang/test/CodeGenCXX/bitfield-access-empty.cpp

  Log Message:
  -----------
  [Clang] Partially fix m68k alignments (#144740)

As the data layout a few lines further up specifies, the int, long and
pointer alignment should be 16 instead of the default of 32.

The long long alignment is also incorrect, but that would require a
change to the data layout as well.

Comparison with GCC, which consistently uses 2 byte alignment:
https://gcc.godbolt.org/z/K3x6a7dEf At least based on some spot checks,
the changes to bit field layout also make use match GCC now.

This was found by https://github.com/llvm/llvm-project/pull/144720.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications