[all-commits] [llvm/llvm-project] 9a8f3b: [clang][RISCV] Set vscale_range attribute based on...

Philip Reames via All-commits all-commits at lists.llvm.org
Wed Oct 19 16:14:57 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9a8f3b113d05d94d9aaacdf511365dde4e35ce4f
      https://github.com/llvm/llvm-project/commit/9a8f3b113d05d94d9aaacdf511365dde4e35ce4f
  Author: Philip Reames <preames at rivosinc.com>
  Date:   2022-10-19 (Wed, 19 Oct 2022)

  Changed paths:
    M clang/lib/Basic/Targets/RISCV.cpp
    M clang/test/CodeGen/riscv-vector-bits-vscale-range.c
    M llvm/include/llvm/Support/RISCVISAInfo.h

  Log Message:
  -----------
  [clang][RISCV] Set vscale_range attribute based on VLEN

Follow up on D135894, restructure code to work in terms of minimum and maximum VLEN coming from RISCVISAInfo.cpp. In the original review, I'd mentioned that MinVLEN was sometimes zero. This turns out to be a case of human error, combined with really bad (lack of) error reporting.

This patch adds appropriate tests for various vector extension combinations to show the mechanism works, but doesn't try to provide exhaustive coverage of the extension interactions. Presumably, that is already covered in existing tests elsewhere.

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




More information about the All-commits mailing list