⚙ D2680 Debug info: Support fragmented variables. (original) (raw)

This is an archive of the discontinued LLVM Phabricator instance.

Table of Contentst-lib/Transforms/Scalar/-Transforms/-Scalar/33SROA.cpp-test/DebugInfo/X86/-DebugInfo/-X86/-array2.ll2sroasplit-1.ll1sroasplit-2.llHide PanelfKeyboard Reference? Differential D2680 Authored by aprantl on Feb 3 2014, 3:16 PM.Download Raw DiffReviewers chandlercaprantl Event TimelineComment ActionsComment ActionsComment ActionsComment Actionsinclude/llvm/DebugInfo.h 684 ↗(On Diff #6836) lib/CodeGen/AsmPrinter/DwarfDebug.cpp 1305 ↗(On Diff #6836) 1320 ↗(On Diff #6836) lib/Transforms/Scalar/SROA.cpp 426 947aprantl updated this revision to Unknown Object (????).Comment ActionsComment ActionsComment ActionsComment ActionsComment Actionslib/Support/GraphWriter.cpp 90 ↗(On Diff #7324) lib/Transforms/Scalar/SROA.cpp 426–429 690 2236 2239 2241–2244 3173Comment ActionsComment ActionsComment ActionsComment ActionsComment ActionsComment ActionsComment Actionslib/Transforms/Scalar/SROA.cpp 970–972 975–976 2107–2108 2116–2117 2124Comment ActionsComment ActionsComment ActionsComment ActionsComment Actionslib/CodeGen/AsmPrinter/DwarfDebug.cpp 1285 ↗(On Diff #12121)dblaikie edited the test plan for this revision. (Show Details)dblaikie removed a reviewer: deleted.Comment Actionslib/Transforms/Scalar/SROA.cpp 970–972 975–976 2107–2108 2116–2117 2124Comment ActionsComment ActionsComment ActionsComment Actionsdblaikie edited the test plan for this revision. (Show Details)dblaikie added a reviewer: chandlerc.dblaikie removed a subscriber: chandlerc.Comment Actionstest/DebugInfo/X86/sroasplit-2.ll 9Comment ActionsComment ActionsComment Actionslib/Transforms/Scalar/SROA.cpp 959 2117–2118 3724Comment ActionsComment Actionslib/Transforms/Scalar/SROA.cpp 2112 2166–2177 3713–3714aprantl added inline comments.lib/Transforms/Scalar/SROA.cpp 2112 2166–2177 3713–3714Comment ActionsComment Actionsaprantl added inline comments.lib/Transforms/Scalar/SROA.cpp 3713–3714Comment ActionsComment Actionslib/Transforms/Scalar/SROA.cpp 3245–3247 3717 test/DebugInfo/X86/sroasplit-1.ll 4aprantl added inline comments.lib/Transforms/Scalar/SROA.cpp 3245–3247 3717 test/DebugInfo/X86/sroasplit-1.ll 4Comment ActionsComment Actionslib/Transforms/Scalar/SROA.cpp 3717Comment ActionsComment ActionsThis revision is now accepted and ready to land.aprantl closed this revision.FilesHistoryCommitsPathSizelib/Transforms/Scalar/SROA.cpp34 linestest/DebugInfo/X86/array2.ll8 linessroasplit-1.ll97 linessroasplit-2.ll102 linesDiffIDBaseDescriptionCreatedLintUnitBaseBaseDiff 16836Feb 3 2014, 3:11 PM★★Diff 27011Added documentation and fixed typos.Feb 11 2014, 3:41 PM★★Diff 37145Rebased patch on ToT and addressed review comments. Most importantly this resolves a conflict with r201190, which added DW_OP_piece support for variables occupying only part of a register (essentially the complementary problem).Feb 14 2014, 7:48 PM★★Diff 47324Updated to also support variables split up by SelectionDAG’s integer type legalization. Thanks iains for providing additional testcases.Feb 24 2014, 2:38 PM★★Diff 57571Mar 5 2014, 3:55 PM★★Diff 67650Here is a rebased version of the patch that applies cleanly after Chandler’s movement of header files and Eric’s creation of AsmPrinterDwarf.Mar 7 2014, 1:24 PM★★Diff 77931Rebased on r204198.Mar 18 2014, 5:53 PM★★Diff 812121Aug 1 2014, 3:42 PM★★Diff 912311Aug 8 2014, 12:41 PM★★Diff 1012313Aug 8 2014, 1:56 PM★★Diff 1115262Oct 22 2014, 11:16 AM★★Diff 1215263Oct 22 2014, 11:21 AM★★Diff 1317303Dec 15 2014, 2:09 PM★★Diff 1417469Dec 18 2014, 1:42 PM★★Diff 1517470Dec 18 2014, 1:45 PM★★Diff 1617472Dec 18 2014, 1:53 PM★★Diff 1717568Dec 22 2014, 12:28 PM★★Diff 1817571Dec 22 2014, 1:29 PM★★This file was added.This file was added.