[clang] [NFC][Clang] Fix Static Code Analysis Concerns with copy without assign (PR #75091)

2023-12-22 Thread via cfe-commits
https://github.com/smanna12 closed https://github.com/llvm/llvm-project/pull/75091 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][Clang] Fix Static Code Analysis Concerns with copy without assign (PR #75091)

2023-12-22 Thread via cfe-commits
smanna12 wrote: > I think the static analysis tool is confused here. The `SVEType` class > doesn't actually have a user-declared copy constructor. It does have a > constructor that delegates to the implicitly declared copy constructor > though. The class members are all scalar types, so the

[clang] [NFC][Clang] Fix Static Code Analysis Concerns with copy without assign (PR #75091)

2023-12-22 Thread Tom Honermann via cfe-commits
https://github.com/tahonermann requested changes to this pull request. I think the static analysis tool is confused here. The `SVEType` class doesn't actually have a user-declared copy constructor. It does have a constructor that delegates to the implicitly declared copy constructor though.

[clang] [NFC][Clang] Fix Static Code Analysis Concerns with copy without assign (PR #75091)

2023-12-11 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (smanna12) Changes This patch adds missing assignment operator to the class which has user-defined copy constructor. --- Full diff: https://github.com/llvm/llvm-project/pull/75091.diff 1 Files Affected: - (modified)

[clang] [NFC][Clang] Fix Static Code Analysis Concerns with copy without assign (PR #75091)

2023-12-11 Thread via cfe-commits
https://github.com/smanna12 created https://github.com/llvm/llvm-project/pull/75091 This patch adds missing assignment operator to the class which has user-defined copy constructor. >From 8772661bda9c93b8bcfeb67f236468c85d1d0253 Mon Sep 17 00:00:00 2001 From: "Manna, Soumi" Date: Mon, 11 Dec