caio/clang-p2996
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[GlobalMerge][PPC] Don't merge globals in llvm.metadata section (#131801)

Browse Source 
The llvm.metadata section is not emitted and has special semantics. We
should not merge globals in it, similarly to how we already skip merging
of `llvm.xyz` globals.

Fixes https://github.com/llvm/llvm-project/issues/131394.
This commit is contained in:
Nikita Popov
2025-04-02 16:40:53 +08:00
committed by GitHub
parent 10c6ebc427
commit 9356091a98
2 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
llvm/lib/CodeGen/GlobalMerge.cpp
View File

@@ -713,7 +713,8 @@ bool GlobalMergeImpl::run(Module &M) {
continue;
// Ignore all 'special' globals.
if (GV.getName().starts_with("llvm.") || GV.getName().starts_with(".llvm."))
if (GV.getName().starts_with("llvm.") ||
GV.getName().starts_with(".llvm.") || Section == "llvm.metadata")
continue;
// Ignore all "required" globals:

9
llvm/test/CodeGen/PowerPC/global-merge-llvm-metadata.ll Normal file
View File

@@ -0,0 +1,9 @@
; RUN: llc -mtriple=powerpc64le-unknown-linux-gnu < %s | FileCheck %s
@index = global i32 0, align 4
@.str = private unnamed_addr constant [1 x i8] zeroinitializer, section "llvm.metadata"
@.str.1 = private unnamed_addr constant [7 x i8] c"test.c\00", section "llvm.metadata"
@llvm.global.annotations = appending global [1 x { ptr, ptr, ptr, i32, ptr }] [{ ptr, ptr, ptr, i32, ptr } { ptr @index, ptr @.str, ptr @.str.1, i32 1, ptr null }], section "llvm.metadata"
; CHECK-NOT: .set
; CHECK-NOT: _MergedGlobals