cor3ntin wrote:
Closing that as an alternative solution was committed
https://github.com/llvm/llvm-project/pull/67089
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/67089
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
topperc wrote:
> Can we reverse this patch? Keep isRVVSizelessBuiltinType and remove
> isRVVType. The implementation of isRVVSizelessBuiltinType is more efficient
> according to some profiling. At least on Release+Asserts build
I went ahead and committed that change.
topperc wrote:
Can we reverse this patch? Keep isRVVSizelessBuiltinType and remove isRVVType.
The implementation of isRVVSizelessBuiltinType is more efficient according to
some profiling. At least on Release+Asserts build
https://github.com/llvm/llvm-project/pull/67089
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/67089
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/eopXD updated https://github.com/llvm/llvm-project/pull/67089
>From 0c449af2049f32103a4f7edf5852f317f6454a23 Mon Sep 17 00:00:00 2001
From: eopXD
Date: Thu, 21 Sep 2023 23:17:22 -0700
Subject: [PATCH] [Clang][RISCV] Remove duplicate functions
isRVVSizelessBuiltinType. NFC
eopXD wrote:
Ping.
https://github.com/llvm/llvm-project/pull/67089
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
eopXD wrote:
Ping.
https://github.com/llvm/llvm-project/pull/67089
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
`isRVVSizelessBuiltinType` and `isRVVType` has the same functionality. This
commit removes the former since we have more variants available in `isRVVType`.
---
Full diff: https://github.com/llvm/llvm-project/pull/67089.diff
5 Files
https://github.com/eopXD created https://github.com/llvm/llvm-project/pull/67089
`isRVVSizelessBuiltinType` and `isRVVType` has the same functionality. This
commit removes the former since we have more variants available in `isRVVType`.
>From 483240e16d800e52783dcfaf52eae81fb1dfca7c Mon Sep 17
10 matches
Mail list logo