[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