testsuite: add testcase for fixed PR107467
Checks
Context |
Check |
Description |
linaro-tcwg-bot/tcwg_gcc_build--master-arm |
success
|
Build passed
|
linaro-tcwg-bot/tcwg_gcc_check--master-arm |
success
|
Test passed
|
linaro-tcwg-bot/tcwg_gcc_build--master-aarch64 |
success
|
Build passed
|
linaro-tcwg-bot/tcwg_gcc_check--master-aarch64 |
success
|
Test passed
|
Commit Message
PR107467 ended up being fixed by the fix for PR115110, but let's
add the testcase on top.
gcc/testsuite/ChangeLog:
PR tree-optimization/107467
PR middle-end/115110
* g++.dg/lto/pr107467_0.C: New test.
---
OK?
gcc/testsuite/g++.dg/lto/pr107467_0.C | 52 +++++++++++++++++++++++++++
1 file changed, 52 insertions(+)
create mode 100644 gcc/testsuite/g++.dg/lto/pr107467_0.C
Comments
Sam James <sam@gentoo.org> writes:
> PR107467 ended up being fixed by the fix for PR115110, but let's
> add the testcase on top.
>
> gcc/testsuite/ChangeLog:
> PR tree-optimization/107467
> PR middle-end/115110
>
> * g++.dg/lto/pr107467_0.C: New test.
> ---
> OK?
>
> gcc/testsuite/g++.dg/lto/pr107467_0.C | 52 +++++++++++++++++++++++++++
> 1 file changed, 52 insertions(+)
> create mode 100644 gcc/testsuite/g++.dg/lto/pr107467_0.C
>
> diff --git a/gcc/testsuite/g++.dg/lto/pr107467_0.C b/gcc/testsuite/g++.dg/lto/pr107467_0.C
> new file mode 100644
> index 000000000000..3a2c1c2d7ae1
> --- /dev/null
> +++ b/gcc/testsuite/g++.dg/lto/pr107467_0.C
> @@ -0,0 +1,52 @@
> +/* { dg-lto-do run } */
> +/* { dg-lto-options { -O2 -fno-strict-aliasing -flto } } */
This should be {{ }} around the options themselves, fixed locally.
> +
> +template <class T>
> +struct pair
> +{
> + int first;
> + T second;
> +};
> +
> +template <typename C>
> +[[gnu::optimize("strict-aliasing")]]
> +bool __attribute__((noinline))
> +compare_pairs(const pair<C> &lhs, const pair<C> &rhs) {
> + return lhs.first == rhs.first && lhs.second == rhs.second;
> +}
> +
> +template <typename B> struct Combined {
> + pair<B *>
> +__attribute__((noinline)) get_const() {
> + return pair<B *>{123, nullptr};
> + }
> +[[gnu::optimize("strict-aliasing")]]
> + bool
> +__attribute__((noinline)) clashy() {
> + return compare_pairs(get_const(), get_const());
> + }
> +};
> +
> +class SomeClass {};
> +class OtherClass {};
> +
> +[[gnu::optimize("strict-aliasing")]]
> +[[gnu::used]]
> +void some_func() {
> + Combined<SomeClass> myvar;
> + __builtin_printf("%i\n", myvar.clashy());
> +}
> +
> +[[gnu::optimize("strict-aliasing")]]
> +void other_func() {
> + Combined<OtherClass> myvar;
> + int t = myvar.clashy();
> + if (!t)
> + __builtin_abort();
> +}
> +
> +[[gnu::optimize("O0")]]
> +int main()
> +{
> + other_func();
> +}
On Oct 25, 2024, at 12:47 PM, Sam James <sam@gentoo.org> wrote:
>
> PR107467 ended up being fixed by the fix for PR115110, but let's
> add the testcase on top.
>
> gcc/testsuite/ChangeLog:
> PR tree-optimization/107467
> PR middle-end/115110
>
> * g++.dg/lto/pr107467_0.C: New test.
> ---
> OK?
Ok.
Mike Stump <mikestump@comcast.net> writes:
> On Oct 25, 2024, at 12:47 PM, Sam James <sam@gentoo.org> wrote:
>>
>> PR107467 ended up being fixed by the fix for PR115110, but let's
>> add the testcase on top.
>>
>> gcc/testsuite/ChangeLog:
>> PR tree-optimization/107467
>> PR middle-end/115110
>>
>> * g++.dg/lto/pr107467_0.C: New test.
>> ---
>> OK?
>
> Ok.
Thanks, pushed.
new file mode 100644
@@ -0,0 +1,52 @@
+/* { dg-lto-do run } */
+/* { dg-lto-options { -O2 -fno-strict-aliasing -flto } } */
+
+template <class T>
+struct pair
+{
+ int first;
+ T second;
+};
+
+template <typename C>
+[[gnu::optimize("strict-aliasing")]]
+bool __attribute__((noinline))
+compare_pairs(const pair<C> &lhs, const pair<C> &rhs) {
+ return lhs.first == rhs.first && lhs.second == rhs.second;
+}
+
+template <typename B> struct Combined {
+ pair<B *>
+__attribute__((noinline)) get_const() {
+ return pair<B *>{123, nullptr};
+ }
+[[gnu::optimize("strict-aliasing")]]
+ bool
+__attribute__((noinline)) clashy() {
+ return compare_pairs(get_const(), get_const());
+ }
+};
+
+class SomeClass {};
+class OtherClass {};
+
+[[gnu::optimize("strict-aliasing")]]
+[[gnu::used]]
+void some_func() {
+ Combined<SomeClass> myvar;
+ __builtin_printf("%i\n", myvar.clashy());
+}
+
+[[gnu::optimize("strict-aliasing")]]
+void other_func() {
+ Combined<OtherClass> myvar;
+ int t = myvar.clashy();
+ if (!t)
+ __builtin_abort();
+}
+
+[[gnu::optimize("O0")]]
+int main()
+{
+ other_func();
+}