ksromanov wrote:
Thanks a lot!
https://github.com/llvm/llvm-project/pull/104540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ksromanov wrote:
> EDIT: Turns out I can just do it, never knew I could press that button on the
> PR. Anyway should re-trigger things :).
Thank you!
https://github.com/llvm/llvm-project/pull/104540
___
cfe-commits mailing list
cfe-commits@lists.llv
@@ -141,6 +141,10 @@ New check aliases
Changes in existing checks
^^
+- Fixed :doc:`cppcoreguidelines-pro-type-union-access
+ ` check to
+ report a location even when the member location is not valid.
ksromanov wrote:
Done.
https://
https://github.com/ksromanov updated
https://github.com/llvm/llvm-project/pull/104540
>From 31ed1cf97be643b3a6c9a05d4e461789c37e2408 Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH 1/7] Workaround for cppcoreguidelines-pro-type-union-access
ksromanov wrote:
All of the issues addressed long time ago.
https://github.com/llvm/llvm-project/pull/104540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ksromanov updated
https://github.com/llvm/llvm-project/pull/104540
>From 31ed1cf97be643b3a6c9a05d4e461789c37e2408 Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH 1/6] Workaround for cppcoreguidelines-pro-type-union-access
ksromanov wrote:
@PiotrZSL would you please review once again?
https://github.com/llvm/llvm-project/pull/104540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ksromanov updated
https://github.com/llvm/llvm-project/pull/104540
>From 31ed1cf97be643b3a6c9a05d4e461789c37e2408 Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH 1/6] Workaround for cppcoreguidelines-pro-type-union-access
@@ -116,6 +116,10 @@ Changes in existing checks
` check to
remove `->`, when redundant `get()` is removed.
+- Fixed :doc:`cppcoreguidelines-pro-type-union-access
+ ` check to
+ report location even when member location is not valid.
ksromanov wrote:
Tha
https://github.com/ksromanov updated
https://github.com/llvm/llvm-project/pull/104540
>From 31ed1cf97be643b3a6c9a05d4e461789c37e2408 Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH 1/6] Workaround for cppcoreguidelines-pro-type-union-access
https://github.com/ksromanov updated
https://github.com/llvm/llvm-project/pull/104540
>From 31ed1cf97be643b3a6c9a05d4e461789c37e2408 Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH 1/5] Workaround for cppcoreguidelines-pro-type-union-access
ksromanov wrote:
> This is still missing a release note, see
Many thanks! This is my first PR to `llvm`, I didn't know about it, sorry.
https://github.com/llvm/llvm-project/pull/104540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
https://github.com/ksromanov updated
https://github.com/llvm/llvm-project/pull/104540
>From 47db38f62940e6bca72a51f519a5b78da25a38c2 Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH 1/4] Workaround for cppcoreguidelines-pro-type-union-access
https://github.com/ksromanov edited
https://github.com/llvm/llvm-project/pull/104540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ksromanov edited
https://github.com/llvm/llvm-project/pull/104540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ksromanov updated
https://github.com/llvm/llvm-project/pull/104540
>From 47db38f62940e6bca72a51f519a5b78da25a38c2 Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH 1/4] Workaround for cppcoreguidelines-pro-type-union-access
https://github.com/ksromanov updated
https://github.com/llvm/llvm-project/pull/104540
>From 5c5f8496341e7b97faf62dad2ba7e0b210bde11b Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH 1/2] Workaround for cppcoreguidelines-pro-type-union-access
https://github.com/ksromanov created
https://github.com/llvm/llvm-project/pull/104540
Fixes #102945
>From 5c5f8496341e7b97faf62dad2ba7e0b210bde11b Mon Sep 17 00:00:00 2001
From: Konstantin Romanov
Date: Fri, 9 Aug 2024 23:40:07 -0400
Subject: [PATCH] Workaround for cppcoreguidelines-pro-type-u
18 matches
Mail list logo
