[all-commits] [llvm/llvm-project] 664701: [RISCV] Don't expose any constructors of RISCVISAI...

[Craig Topper via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 66470112d6e00832cae2970fb56f491cd5f7b14a
      https://github.com/llvm/llvm-project/commit/66470112d6e00832cae2970fb56f491cd5f7b14a
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2024-07-09 (Tue, 09 Jul 2024)

  Changed paths:
    M lld/ELF/Arch/RISCV.cpp
    M llvm/include/llvm/TargetParser/RISCVISAInfo.h
    M llvm/lib/TargetParser/RISCVISAInfo.cpp

  Log Message:
  -----------
  [RISCV] Don't expose any constructors of RISCVISAInfo publicly. (#98249)

lld was using RISCVISAInfo(unsigned XLen,
RISCVISAUtils::OrderedExtensionMap &Exts). This required a call to
RISCVISAInfo::postProcessAndChecking to validate the RISCVISAInfo that
was created. This exposes too much about RISCVISAInfo to lld.

Replace with a new RISCVISAInfo::createFromExtMap that is responsible
for creating the object and calling postProcessAndChecking.



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