[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2024-05-03 Thread via cfe-commits
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

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2024-05-03 Thread via 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

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2023-12-08 Thread Craig Topper via 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.

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2023-12-08 Thread Craig Topper via 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 https://github.com/llvm/llvm-project/pull/67089

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2023-11-15 Thread Craig Topper via cfe-commits
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

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2023-11-15 Thread Yueh-Ting Chen via 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

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2023-11-15 Thread Yueh-Ting Chen via 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

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2023-10-01 Thread Yueh-Ting Chen via 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

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2023-09-22 Thread via 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

[clang] [Clang][RISCV] Remove duplicate functions isRVVSizelessBuiltinType. NFC (PR #67089)

2023-09-22 Thread Yueh-Ting Chen via cfe-commits
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