[clang] [Clang] Swap range metadata to attribute for intrinsics. (PR #94851)

https://github.com/andjo403 closed https://github.com/llvm/llvm-project/pull/94851 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Swap range metadata to attribute for intrinsics. (PR #94851)

andjo403 wrote: also changed the noundef to mach the range attribute https://github.com/llvm/llvm-project/pull/94851 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Swap range metadata to attribute for intrinsics. (PR #94851)

https://github.com/andjo403 updated https://github.com/llvm/llvm-project/pull/94851 >From 6b4556ea373d90a780c132ab2c51ae46d40a3f49 Mon Sep 17 00:00:00 2001 From: Andreas Jonson Date: Sun, 26 May 2024 10:02:25 +0200 Subject: [PATCH 1/2] [Clang] swap range metadata to attribute for Intrinsics.

[clang] [Clang] Swap range metadata to attribute for intrinsics. (PR #94851)

https://github.com/andjo403 created https://github.com/llvm/llvm-project/pull/94851 Continues to swap out range metadata to range attribute for calls to be able to deprecate range metadata on calls in the future >From 6b4556ea373d90a780c132ab2c51ae46d40a3f49 Mon Sep 17 00:00:00 2001 From:

[clang] [llvm] [inline] Clone return range attribute on the callsite into inlined call (PR #92666)

andjo403 wrote: Comment fixed and need help with merging. https://github.com/llvm/llvm-project/pull/92666 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [inline] Clone return range attribute on the callsite into inlined call (PR #92666)

@@ -1444,6 +1445,8 @@ static AttrBuilder IdentifyValidPoisonGeneratingAttributes(CallBase ) { Valid.addAttribute(Attribute::NonNull); if (CB.hasRetAttr(Attribute::Alignment)) Valid.addAlignmentAttr(CB.getRetAlign()); + if (CB.hasRetAttr(Attribute::Range)) +

[clang] [llvm] [inline] Clone return range attribute on the callsite into inlined call (PR #92666)

https://github.com/andjo403 updated https://github.com/llvm/llvm-project/pull/92666 >From 1bc922a6797555b0bf69b187224ba054978fb7ad Mon Sep 17 00:00:00 2001 From: Andreas Jonson Date: Sat, 18 May 2024 19:42:03 +0200 Subject: [PATCH 1/2] [inline] Tests for clone return range attribute on the

[clang] [llvm] [inline] Clone return range attribute on the callsite into inlined call (PR #92666)

andjo403 wrote: It was the usage of exactIntersectWith in https://github.com/llvm/llvm-project/pull/91101 vs the intersectWith that I used that I wanted to sort out but think that we have conluded that it shall be intersectWith. so this it ready for review

[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)

@@ -1427,8 +1429,17 @@ static void AddParamAndFnBasicAttributes(const CallBase , ValidExactParamAttrs[ArgNo].getAlignment().valueOrOne()) AL = AL.removeParamAttribute(Context, I, Attribute::Alignment); + auto ExistingRange =

[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)

https://github.com/andjo403 edited https://github.com/llvm/llvm-project/pull/91101 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)

@@ -1427,8 +1429,20 @@ static void AddParamAndFnBasicAttributes(const CallBase , ValidExactParamAttrs[ArgNo].getAlignment().valueOrOne()) AL = AL.removeParamAttribute(Context, I, Attribute::Alignment); + auto ExistingRange =

[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)

@@ -1427,8 +1429,20 @@ static void AddParamAndFnBasicAttributes(const CallBase , ValidExactParamAttrs[ArgNo].getAlignment().valueOrOne()) AL = AL.removeParamAttribute(Context, I, Attribute::Alignment); + auto ExistingRange =

[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)

@@ -1427,8 +1429,20 @@ static void AddParamAndFnBasicAttributes(const CallBase , ValidExactParamAttrs[ArgNo].getAlignment().valueOrOne()) AL = AL.removeParamAttribute(Context, I, Attribute::Alignment); + auto ExistingRange =

[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)

@@ -1427,8 +1429,20 @@ static void AddParamAndFnBasicAttributes(const CallBase , ValidExactParamAttrs[ArgNo].getAlignment().valueOrOne()) AL = AL.removeParamAttribute(Context, I, Attribute::Alignment); + auto ExistingRange =

[clang] [llvm] [inline] Clone return range attribute on the callsite into inlined call (PR #92666)

andjo403 wrote: hmm noticed #91101 now looks like I need to coordinate with that PR https://github.com/llvm/llvm-project/pull/92666 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [inline] Clone return range attribute on the callsite into inlined call (PR #92666)

https://github.com/andjo403 created https://github.com/llvm/llvm-project/pull/92666 CC @goldsteinn @nikic >From 6dd513a670e813a5e6745044bc69fdd7a7b7c4d9 Mon Sep 17 00:00:00 2001 From: Andreas Jonson Date: Sat, 18 May 2024 19:42:03 +0200 Subject: [PATCH 1/2] [inline] Tests for clone return

[clang] [llvm] [SCCP] Swap out range metadata to range attribute (PR #90134)

andjo403 wrote: CC @nikic as you have looked at the most of other range attribute PRs https://github.com/llvm/llvm-project/pull/90134 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [SCCP] Swap out range metadata to range attribute (PR #90134)

https://github.com/andjo403 created https://github.com/llvm/llvm-project/pull/90134 Also moved the range from the function's call sites to the functions return value as that is possible now. >From f0c948df7b158235d8631186f54a31b4e7c0bdf1 Mon Sep 17 00:00:00 2001 From: Andreas Jonson Date:

[clang] [llvm] [InstCombine] Swap out range metadata to range attribute for cttz/ctlz/ctpop (PR #88776)

andjo403 wrote: I still have not asked for rights to merge do not know when there is a good time to do that feels like there is a lot of discussions about if it is needed to have it or not. https://github.com/llvm/llvm-project/pull/88776 ___

[clang] [llvm] [InstCombine] Swap out range metadata to range attribute for cttz/ctlz/ctpop (PR #88776)

andjo403 wrote: This time it seems like I have managed to get all the tests to pass at least on linux, windows have not executed yet. https://github.com/llvm/llvm-project/pull/88776 ___ cfe-commits mailing list cfe-commits@lists.llvm.org