[all-commits] [llvm/llvm-project] 1a803e: [12/15][Clang][RISCV][NFC] Refine the way to check...

Yueh-Ting (eop) Chen via All-commits all-commits at lists.llvm.org
Tue Jan 24 08:28:16 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1a803e0b2ad7ea9cfe7fc832210f537fce3f5a51
      https://github.com/llvm/llvm-project/commit/1a803e0b2ad7ea9cfe7fc832210f537fce3f5a51
  Author: eopXD <yueh.ting.chen at gmail.com>
  Date:   2023-01-24 (Tue, 24 Jan 2023)

  Changed paths:
    M clang/include/clang/Basic/riscv_vector.td
    M clang/include/clang/Support/RISCVVIntrinsicUtils.h
    M clang/lib/CodeGen/CGBuiltin.cpp

  Log Message:
  -----------
  [12/15][Clang][RISCV][NFC] Refine the way to check for Policy in riscv_vector_builtin_cg.inc

The current way creates a fallacy that checking for
`PolicyAttrs == TAIL_AGNOSTIC` is implicitly equivalant to
`TAIL_AGNOSTIC_MASK_UNDISTURBED`. This works under the assumption that
an unmasked intrinsic has a policy of TAMU. The expression here is
mis-leading and will not be correct when the default policy is not
TAMU.

As this patch-set targets to change the default policy from TAMU to
TAMA, this commit is necessary before changing the default.

This is the 12th commit of a patch-set that aims to change the default policy
for RVV intrinsics from TAMU to TAMA.

Please refer to the cover letter in the 1st commit (D141573) for an
overview.

Reviewed By: craig.topper

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




More information about the All-commits mailing list