[all-commits] [llvm/llvm-project] 7013a7: [Clang] Limit FunctionTypeExtraBitfields::NumExcep...

sdesmalen-arm via All-commits all-commits at lists.llvm.org
Tue Jun 6 05:10:46 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7013a751f170d829b35e7bb153d3334d144d5d54
      https://github.com/llvm/llvm-project/commit/7013a751f170d829b35e7bb153d3334d144d5d54
  Author: Sander de Smalen <sander.desmalen at arm.com>
  Date:   2023-06-06 (Tue, 06 Jun 2023)

  Changed paths:
    M clang/include/clang/AST/Type.h
    M clang/lib/AST/Type.cpp

  Log Message:
  -----------
  [Clang] Limit FunctionTypeExtraBitfields::NumExceptionType to 16 bits.

In https://reviews.llvm.org/D127762#4102578 @erichkeane suggested to
limit size of this field to 16bits, such that the field that encodes the
SME attributes for a function fall within the alignment of the struct for
32bit platforms.

Standard implimits defines the minimum handlers per try block to 256,
which suggests that 16bits should be more than sufficient for most
programs. Erich also pointed out that exception specs are being
deprecated and are rarely used, so hopefully this change is safe to make.

Reviewed By: erichkeane

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




More information about the All-commits mailing list