[all-commits] [llvm/llvm-project] af602e: [RISCV] Make RISCVISAInfo::toFeatureVector ignore ...

Alex Bradbury via All-commits all-commits at lists.llvm.org
Sun Mar 26 07:52:25 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: af602edf0ecb4e1d7de4ccce8ebe1be8bb53443d
      https://github.com/llvm/llvm-project/commit/af602edf0ecb4e1d7de4ccce8ebe1be8bb53443d
  Author: Alex Bradbury <asb at igalia.com>
  Date:   2023-03-26 (Sun, 26 Mar 2023)

  Changed paths:
    M llvm/lib/Support/RISCVISAInfo.cpp
    M llvm/unittests/Support/RISCVISAInfoTest.cpp

  Log Message:
  -----------
  [RISCV] Make RISCVISAInfo::toFeatureVector ignore unsupported extensions

parseNormalizedArchString adds a code path that creates a RISCVISAInfo
including extensions that may not be supported by LLVM (rather than
erroring or just ignoring them). Therefore, toFeatureVector needs to
check the extension is supported in order to avoid creating unrecognised
feature strings.

This change shouldn't impact any code paths used outside of test code,
but this will be relied upon by the next patch which moves llvm-objdump
and related tools over to using parseNormalizedArchString.

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




More information about the All-commits mailing list