[PATCH] D149135: [Assignment Tracking] Remove overly defensive AllocaInst assertion

Stephen Tozer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 02:41:16 PDT 2023


StephenTozer accepted this revision.
StephenTozer added a comment.
This revision is now accepted and ready to land.

Description seems clear enough, changing the assert seems reasonable, LGTM with the tiniest nit.



================
Comment at: llvm/test/DebugInfo/assignment-tracking/X86/global-storage.ll:17-24
+define dso_local void @_Z3funi(i32 noundef %x) #0 !dbg !15 {
+entry:
+  store i32 %x, ptr @a, align 4, !DIAssignID !27
+  call void @llvm.dbg.assign(metadata i32 %x, metadata !23, metadata !DIExpression(), metadata !27, metadata ptr @a, metadata !DIExpression()), !dbg !21
+  ret void
+}
+
----------------



CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149135/new/

https://reviews.llvm.org/D149135



More information about the llvm-commits mailing list