[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/133481 >From fde8c9e4834a2dad33c1349ef94fc90544a09b65 Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Fri, 21 Mar 2025 16:49:14 + Subject: [PATCH 1/2] [KeyInstr] Inline atom info Source atom groups are

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/133481 >From fde8c9e4834a2dad33c1349ef94fc90544a09b65 Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Fri, 21 Mar 2025 16:49:14 + Subject: [PATCH] [KeyInstr] Inline atom info Source atom groups are iden

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

https://github.com/SLTozer approved this pull request. LGTM, besides a couple inline comments. https://github.com/llvm/llvm-project/pull/133481 ___ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailm

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

@@ -1813,7 +1813,8 @@ static DebugLoc inlineDebugLoc(DebugLoc OrigDL, DILocation *InlinedAt, DenseMap &IANodes) { auto IA = DebugLoc::appendInlinedAt(OrigDL, InlinedAt, Ctx, IANodes); return DILocation::get(Ctx, OrigDL.getLine(), OrigDL.getCo

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

@@ -0,0 +1,53 @@ +; RUN: opt %s -passes=inline -S -o - | FileCheck %s + +;; Inline `f` into `g`. The inlined assignment store and add should retain +;; their atom info. + +; CHECK: _Z1gi +; CHECK-NOT: _Z1fi +; CHECK: %add.i = add nsw i32 %mul.i, 1, !dbg [[G1R2:!.*]] +; CHECK-NEXT:

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

https://github.com/SLTozer edited https://github.com/llvm/llvm-project/pull/133481 ___ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

llvmbot wrote: @llvm/pr-subscribers-debuginfo Author: Orlando Cazalet-Hyams (OCHyams) Changes Source atom groups are identified by an atom group number and inlined-at pair, so we simply can copy the atom numbers into the caller when inlining. --- Full diff: https://github.com/llvm/llvm-pr

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

https://github.com/OCHyams ready_for_review https://github.com/llvm/llvm-project/pull/133481 ___ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

OCHyams wrote: > [!WARNING] > This pull request is not mergeable via GitHub because a downstack PR is > open. Once all requirements are satisfied, merge this PR as a stack href="https://app.graphite.dev/github/pr/llvm/llvm-project/133481?utm_source=stack-comment-downstack-mergeability-warning";

[llvm-branch-commits] [llvm] [KeyInstr] Inline atom info (PR #133481)

https://github.com/OCHyams created https://github.com/llvm/llvm-project/pull/133481 Source atom groups are identified by an atom group number and inlined-at pair, so we simply can copy the atom numbers into the caller when inlining. >From a6f7a923dde98dadbb2735c8ec4cde83d0353d3f Mon Sep 17 00:0