[all-commits] [llvm/llvm-project] 8aceb7: [ValueTypes] Remove MVT::MAX_ALLOWED_VALUETYPE. NF...

Craig Topper via All-commits all-commits at lists.llvm.org
Wed May 29 11:47:44 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8aceb7a53d82f9566a7cf4775effa4089b22a75b
      https://github.com/llvm/llvm-project/commit/8aceb7a53d82f9566a7cf4775effa4089b22a75b
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2024-05-29 (Wed, 29 May 2024)

  Changed paths:
    M llvm/include/llvm/CodeGenTypes/MachineValueType.h
    M llvm/lib/CodeGen/SelectionDAG/LegalizeTypes.h
    M llvm/lib/CodeGen/TargetLoweringBase.cpp

  Log Message:
  -----------
  [ValueTypes] Remove MVT::MAX_ALLOWED_VALUETYPE. NFC (#93654)

Despite the comment, this isn't used to size bit vectors or tables.
That's done by VALUETYPE_SIZE. MAX_ALLOWED_VALUETYPE is only used by
some static_asserts that compare it to VALUETYPE_SIZE.

This patch removes it and most of the static_asserts. I left one where I
compared VALUETYPE_SIZE to token which is the first type that isn't part
of the VALUETYPE range. This isn't strictly needed, we'd probably catch
duplication error from VTEmitter.cpp first.



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


More information about the All-commits mailing list