-
Notifications
You must be signed in to change notification settings - Fork 11k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[BasicBlockUtils] Remove redundant llvm.dbg instructions after blocks…
… to reduce compile time (#89069) this patch is to fix the compile time for some cases, before this change, some targets (riscv-64, ve) will spend much more compile time on this case (https://godbolt.org/z/rrov17cTo). With this change, the compile time was reduced a lot. Fixes #89073 PR: #89069
- Loading branch information
1 parent
213ab96
commit 2e3e086
Showing
4 changed files
with
49 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
45 changes: 45 additions & 0 deletions
45
llvm/test/Transforms/LoopUnroll/unroll-remove-redundant-dbg.ll
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,45 @@ | ||
; RUN: opt < %s -S -passes=loop-unroll | FileCheck %s | ||
|
||
define i64 @d(i1 %tobool.not, i32 %add, i64 %conv23) !dbg !14{ | ||
entry: | ||
br label %for.body | ||
|
||
for.body: ; preds = %for.body, %entry | ||
; There should be only one "llvm.dbg.vale" after loop unrolling | ||
; CHECK: call void @llvm.dbg.value | ||
; CHECK-NOT: call void @llvm.dbg.value | ||
|
||
%k.045 = phi i64 [ 0, %entry ], [ %k.046, %for.body ] | ||
tail call void @llvm.dbg.value(metadata i32 0, metadata !13, metadata !DIExpression()), !dbg !17 | ||
%k.046 = add nuw nsw i64 %k.045, 1 | ||
%exitcond = icmp ne i64 %k.046, 5 | ||
br i1 %exitcond, label %for.body, label %for.end22 | ||
|
||
for.end22: ; preds = %for.body | ||
ret i64 %k.046 | ||
} | ||
|
||
; Function Attrs: nocallback nofree nosync nounwind speculatable willreturn memory(none) | ||
declare void @llvm.dbg.value(metadata, metadata, metadata) | ||
|
||
!llvm.dbg.cu = !{!0} | ||
!llvm.module.flags = !{!12} | ||
|
||
!0 = distinct !DICompileUnit(language: DW_LANG_C89, file: !1, producer: "clang version 19.0.0git (https://github.com/llvm/llvm-project.git ec062f5b33ed22c61742e3c1486f6cba915801e0)", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, globals: !2, splitDebugInlining: false, nameTableKind: None) | ||
!1 = !DIFile(filename: "unroll-remove-redundant-dbg.c", directory: "", checksumkind: CSK_MD5, checksum: "aa30a1d8c04deb9b0f3885c258d2b674") | ||
!2 = !{!3, !8, !10} | ||
!3 = !DIGlobalVariableExpression(var: !4, expr: !DIExpression()) | ||
!4 = distinct !DIGlobalVariable(name: "a", scope: !0, file: !1, line: 2, type: !5, isLocal: false, isDefinition: true) | ||
!5 = !DIDerivedType(tag: DW_TAG_typedef, name: "uint32_t", file: !6, line: 198, baseType: !7) | ||
!6 = !DIFile(filename: "/usr/include/stdint.h", directory: "", checksumkind: CSK_MD5, checksum: "da031bcff2d0c1d65aa92e7e68a44ef3") | ||
!7 = !DIBasicType(name: "unsigned int", size: 32, encoding: DW_ATE_unsigned) | ||
!8 = !DIGlobalVariableExpression(var: !9, expr: !DIExpression()) | ||
!9 = distinct !DIGlobalVariable(name: "c", scope: !0, file: !1, line: 2, type: !5, isLocal: false, isDefinition: true) | ||
!10 = !DIGlobalVariableExpression(var: !11, expr: !DIExpression()) | ||
!11 = distinct !DIGlobalVariable(name: "b", scope: !0, file: !1, line: 2, type: !5, isLocal: false, isDefinition: true) | ||
!12 = !{i32 2, !"Debug Info Version", i32 3} | ||
!13 = !DILocalVariable(name: "f", scope: !14, file: !1, line: 4, type: !5) | ||
!14 = distinct !DISubprogram(name: "d", scope: !1, file: !1, line: 3, type: !15, scopeLine: 3, flags: DIFlagAllCallsDescribed, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !0, retainedNodes: !16) | ||
!15 = !DISubroutineType(types: !16) | ||
!16 = !{} | ||
!17 = !DILocation(line: 0, scope: !14) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters