[PATCH] D117489: [SDAG] Don't move DBG_VALUE instructions after insertion point during scheduling (PR53243)

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 22 14:40:46 PST 2022


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

LGTM, thanks



================
Comment at: llvm/test/CodeGen/X86/pr53243-tail-call-fastisel.ll:29
+!5 = !DIFile(filename: "<unknown>", directory: "")
+!6 = !DINamespace(name: "option", scope: !7)
+!7 = !DINamespace(name: "core", scope: null)
----------------
nit: most of those namespaces/enumerates should not be needed I think.


================
Comment at: llvm/test/CodeGen/X86/pr53243-tail-call-fastisel.ll:43
+!19 = !DIDerivedType(tag: DW_TAG_member, name: "data_ptr", scope: !17, file: !5, baseType: !20, size: 64, align: 64)
+!20 = !DIDerivedType(tag: DW_TAG_pointer_type, name: "*const u8", baseType: !8, size: 64, align: 64, dwarfAddressSpace: 0)
+!21 = !DIDerivedType(tag: DW_TAG_member, name: "length", scope: !17, file: !5, baseType: !22, size: 64, align: 64, offset: 64)
----------------
nit: most of the types shouldn't be needed.


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

https://reviews.llvm.org/D117489



More information about the llvm-commits mailing list