[all-commits] [llvm/llvm-project] 669d2b: [Clang][LoongArch] Pre-commit test for D156116

Lu Weining via All-commits all-commits at lists.llvm.org
Mon Jul 24 18:00:18 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 669d2bbc837ee56d6bb05d5d55615246fc8b6b5c
      https://github.com/llvm/llvm-project/commit/669d2bbc837ee56d6bb05d5d55615246fc8b6b5c
  Author: Weining Lu <luweining at loongson.cn>
  Date:   2023-07-25 (Tue, 25 Jul 2023)

  Changed paths:
    A clang/test/CodeGen/LoongArch/abi-lp64d-empty-structs.c

  Log Message:
  -----------
  [Clang][LoongArch] Pre-commit test for D156116

Differential Revision: https://reviews.llvm.org/D156114


  Commit: c299efbf284fae92bb8d7d29110f76912f089daf
      https://github.com/llvm/llvm-project/commit/c299efbf284fae92bb8d7d29110f76912f089daf
  Author: Weining Lu <luweining at loongson.cn>
  Date:   2023-07-25 (Tue, 25 Jul 2023)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/lib/CodeGen/Targets/LoongArch.cpp
    M clang/test/CodeGen/LoongArch/abi-lp64d-empty-structs.c

  Log Message:
  -----------
  [Clang][LoongArch] Fix ABI handling of empty structs in C++ to match GCC behaviour

GCC doesn't ignore non-zero-length array of empty structures in C++
while clang does. What this patch did is to match GCC's behaviour
although this rule is not documented in psABI.

Similar to D142327 for RISCV.

Reviewed By: xry111, xen0n

Differential Revision: https://reviews.llvm.org/D156116


Compare: https://github.com/llvm/llvm-project/compare/c8bcc48af628...c299efbf284f


More information about the All-commits mailing list