)]}' { "log": [ { "commit": "882e6317dc66e5b241c535a7c8feedd1a440c518", "tree": "4b825dc642cb6eb9a060e54bf8d69288fbee4904", "parents": [ "24369beb1bf2ec871bc9254961d6f64e4e98ebe0" ], "author": { "name": "James Henderson", "email": "james.henderson@sony.com", "time": "Mon Feb 08 15:40:55 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jun 29 10:53:02 2021 -0700" }, "message": "[RFC][debuginfo-test] Rename debug-info lit tests for general purposes\n\nDiscussion thread:\nhttps://lists.llvm.org/pipermail/llvm-dev/2021-January/148048.html\n\nMove debuginfo-test into a subdirectory of a new top-level directory,\ncalled cross-project-tests. The new name replaces \"debuginfo-test\" as an\nLLVM project enabled via LLVM_ENABLE_PROJECTS.\n\nDifferential Revision: https://reviews.llvm.org/D95339\n\nReviewed by: aprantl\n\nGitOrigin-RevId: 1364750dadbb56032ef73b4d0d8cbc88a51392da\n" }, { "commit": "24369beb1bf2ec871bc9254961d6f64e4e98ebe0", "tree": "f562ae6a32ad7a34b268db6cf7c66885d7dd0fcd", "parents": [ "cc6544e56a780cdf9a981d1db5744d549db657a2" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Tue May 25 13:36:11 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:35:01 2021 -0700" }, "message": "[Dexter] Remove erroneously added diff file\n\nDelete d.diff from debuginfo-tests/dexter directory.\n\nGitOrigin-RevId: fc0acd10c0ab43986303de4167e31b46f13367c2\n" }, { "commit": "cc6544e56a780cdf9a981d1db5744d549db657a2", "tree": "37dfd1e9b6682ffe0cbf9e4cdd15ecb8a9a90335", "parents": [ "7b7f2cc54fa08ad1ad521817c728d30ccc7fbac9" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Tue May 25 13:09:14 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:35:00 2021 -0700" }, "message": "[dexter] Change --source-root-dir and add --debugger-use-relative-paths\n\nWe want to use `DexDeclareFile` to specify paths relative to a project root\ndirectory. The option `--source-root-dir`, prior to this patch, causes dexter\nto strip the path prefix from commands before passing them to a debugger, and\nappends the prefix to file paths returned from a debugger. This patch changes\nthe behviour of `--source-root-dir`. Relative paths in commands, made possible\nwith `DexDeclareFile(relative/path)`, are appended to the `--source-root-dir`\ndirectory.\n\nA new option, `--debugger-use-relative-paths`, can be used alongside\n`--source-root-dir` to reproduce the old behaviour: all paths passed to the\ndebugger will be made relative to `--source-root-dir`.\n\nI\u0027ve added a regression test source_root_dir.dex for this new behaviour, and\nmodified the existing `--source-root-dir` regression and unit tests to use\n`--debugger-use-relative-paths`.\n\nReviewed By: jmorse\n\nDifferential Revision: https://reviews.llvm.org/D100307\n\nGitOrigin-RevId: 4b55102aff29f5ce82b38a9e4a819b959e29ecd7\n" }, { "commit": "7b7f2cc54fa08ad1ad521817c728d30ccc7fbac9", "tree": "5fbb45a06cc1a795e39666852730bb8cdadf21a6", "parents": [ "c59622d34fb901ea646a979b570c862067dd5db9" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Tue May 25 12:47:16 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:56 2021 -0700" }, "message": "[Dexter] Add DexDeclareFile command to Dexter\n\nDexDeclareFile allows test producers to write test files with .dex extensions\nthat contain pure dexter commands.\n\n.dex file commands do not need to be commented out like they do when written\ninline within test source files.\n\nDexDeclareFile commands are declarative in behaviour, they state that any\nDexter command seen from this point on will have its path attribute set to the\npath declared in the DexDeclareFile command.\n\nDifferential Revision: https://reviews.llvm.org/D99651\n\nGitOrigin-RevId: c2c2be44ed644199a5a9832bf9ac34fc3ef6b486\n" }, { "commit": "c59622d34fb901ea646a979b570c862067dd5db9", "tree": "63bc7027d66eceaa8f4400fdd580cdc422783246", "parents": [ "23d588a79b3db2ea676fba20354eb0821855a757" ], "author": { "name": "James Henderson", "email": "james.henderson@sony.com", "time": "Tue May 18 10:57:32 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:53 2021 -0700" }, "message": "[debuginfo-tests] Stop using installed LLDB and remove redundancy\n\nThe removed code just replicated what use_llvm_tool does, plus looked\nfor an installed LLDB on the PATH to use. In a monorepo world, it seems\nlikely that if people want to run the tests that require LLDB, they\nshould enable and build LLDB itself. If users really want to use the\ninstalled LLDB executable, they can specify the path to the executable\nas an environment variable \"LLDB\".\n\nSee the discussion in https://reviews.llvm.org/D95339#2638619 for\nmore details.\n\nReviewed by: jmorse, aprantl\n\nDifferential Revision: https://reviews.llvm.org/D102680\n\nGitOrigin-RevId: 5c4a5daf293c1c924caa3c6faed4487682d70045\n" }, { "commit": "23d588a79b3db2ea676fba20354eb0821855a757", "tree": "72769a12857ffeb8d331d03151fa39b727721d0b", "parents": [ "7669eeb4a0e4ca8603de94cefb47f1a89defcf62" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Fri May 21 13:48:31 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:49 2021 -0700" }, "message": "[dexter] Add REQUIRES: lldb to the test label_offset.cpp\n\nThe test feature_tests/subtools/test/label_offset.cpp was added in 723a8ae5\nfor D101147.\n\nGitOrigin-RevId: 35a7c4b4b1d0745775f79d7e5d946e782d923712\n" }, { "commit": "7669eeb4a0e4ca8603de94cefb47f1a89defcf62", "tree": "10857152961c6fc836359812589d5f82c95807cc", "parents": [ "da3d68284ade83df4b60655fa84a55ad9144c856" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Fri May 21 12:43:13 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:46 2021 -0700" }, "message": "[dexter] Add hit_count keyword arg to DexLimitSteps\n\nThe DexLimitSteps command leading breakpoint will be deleted after triggering\n\u0027hit_count\u0027 number of times if the argument is provided.\n\nAll the lit tests pass on linux (with lldb), and I\u0027ve tested this on windows\n(with vs2017) manually as the lit tests for DexLimitSteps are currently\nunsupported on windows.\n\nReviewed By: StephenTozer, chrisjackson\n\nDifferential Revision: https://reviews.llvm.org/D101527\n\nGitOrigin-RevId: 469833f418e1704331f8dccb06ccd391f8d726e1\n" }, { "commit": "da3d68284ade83df4b60655fa84a55ad9144c856", "tree": "05c32a544e3adb2d74ebba5d09af4ed39a47e6d2", "parents": [ "91a959a710577685e1f06a72f35354416318531a" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Fri May 21 09:06:43 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:42 2021 -0700" }, "message": "[dexter] Remove accidentally committed .rej file\n\nThe file was added in 98e7911f9517.\n\nGitOrigin-RevId: 2348b5c943198ffedb3494b8443ae172ba96c00e\n" }, { "commit": "91a959a710577685e1f06a72f35354416318531a", "tree": "ca710baa256ca2ada84a499811070f4438de2119", "parents": [ "238d35a93140488b73dc57b0c0c73b3c3d0f8dfa" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Fri May 21 08:56:14 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:41 2021 -0700" }, "message": "[dexter] Change line label reference syntax to enable label-relative offsets (2/2)\n\nUpdate dexter tests to use the new line label reference syntax introduced in D101147.\n\nUpdated with:\nhttps://gist.github.com/OCHyams/8255efe7757cac266440ed2ba55f1442\n\nReviewed By: chrisjackson, jmorse\n\nDifferential Revision: https://reviews.llvm.org/D101148\n\nGitOrigin-RevId: 98e7911f9517cd9eab0f566ca445096d9e206a75\n" }, { "commit": "238d35a93140488b73dc57b0c0c73b3c3d0f8dfa", "tree": "a8defdd1d1b1a98d57f165037662a3c672ff899b", "parents": [ "02d981d880ae72cb8a447581b3e6d4ce36255e13" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Fri May 21 08:35:17 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:38 2021 -0700" }, "message": "[dexter] Change line label reference syntax to enable label-relative offsets (1/2)\n\nThis patch changes how line labels are resolved in order to enable\nlabel-relative offsets to be used in commands. This is a breaking change in\ndexter. Instead of using label references directly as argument values, labels\nwill instead be referenced through a function `ref(str)`.\n\n // No way to use offsets currently.\n Currently: DexExpectWatchValue(\u0027x\u0027, \u00271\u0027, on_line\u003d\u0027labled_line\u0027)\n Patched: DexExpectWatchValue(\u0027x\u0027, \u00271\u0027, on_line\u003dref(\u0027labled_line\u0027))\n Patched: DexExpectWatchValue(\u0027x\u0027, \u00271\u0027, on_line\u003dref(\u0027labled_line\u0027) + 3)\n\nA dexter command is \"parsed\" by finding the whole command and sending it off to\n`eval`. This change adds a function called `ref` to the `eval` globals map that\nsimply looks up the name and returns an int. If the line name hasn\u0027t been\ndefined, or a name is defined more than once, an error is reported (see\nerr_bad_label_ref.cpp and err_duplicate_label.cpp). Label offsets can be\nachieved by simply writing the desired expression.\n\nThe rationale behind removing the existing label referencing mechanic is for\nconsistency and to simplify the code required to make labels work.\n\nI\u0027ve separated the update to llvm\u0027s dexter tests into another patch for ease of\nreview here (D101148). Here is a small python script which can be used to\nupdate tests to use the new syntax:\nhttps://gist.github.com/OCHyams/8255efe7757cac266440ed2ba55f1442\n\nIf it helps anyone using dexter on downstream tests we can come up with a\ndeprecation plan instead out outright removing the existing syntax.\n\nReviewed By: jmorse\n\nDifferential Revision: https://reviews.llvm.org/D101147\n\nGitOrigin-RevId: 723a8ae5dab25b329c6aebc25564b3630e8e7d17\n" }, { "commit": "02d981d880ae72cb8a447581b3e6d4ce36255e13", "tree": "e0e6378c46565a73341f5a020024078f549ea909", "parents": [ "4e80adf12ca13c251aa6734fb99d8c905a1a118a" ], "author": { "name": "James Henderson", "email": "james.henderson@sony.com", "time": "Thu May 06 09:44:36 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:34 2021 -0700" }, "message": "[debuginfo-tests] Fix environment variable used to specify LLDB\n\nCurrently, if the user specifies the environment variable \u0027CLANG\u0027, tests\nwill attempt to use the value as a path to the clang executable.\nPreviously, lldb could also be specified via the CLANG environment\nvariable, but this was almost certainly a bug, because that meant both\nclang and lldb would have the same path. This patch changes the\nenvironment variable for lldb to \u0027LLDB\u0027.\n\nReviewed by: thopre, teemperor\n\nDifferential Revision: https://reviews.llvm.org/D101982\n\nGitOrigin-RevId: d05ae0fd1732507bcf5ad3dbd0ba2ca77f22f633\n" }, { "commit": "4e80adf12ca13c251aa6734fb99d8c905a1a118a", "tree": "b40d123add4c36f8617ee2e783067a53081270bb", "parents": [ "b8a1d90d4ebbf982fe7e1c6303c4b8a8bb87fddb" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Mon May 17 09:07:46 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:31 2021 -0700" }, "message": "[dexter] Remove requirement for a condition in DexLimitSteps\n\nCurrently the DexLimitSteps command requires at least one condition. This patch\nlets users elide the condition to specify that the breakpoint range should\nalways be activated when the leading line is stepped on. This patch also\nupdates the terminology used in the `ConditionalController` class from the\nterms \u0027conditional\u0027 and \u0027unconditional\u0027 to \u0027leading\u0027 and \u0027trailing\u0027 when\nreferring to the breakpoints in the DexLimitSteps range because the leading\nbreakpoint can now be unconditional.\n\nReviewed By: chrisjackson\n\nDifferential Revision: https://reviews.llvm.org/D101438\n\nGitOrigin-RevId: e35a5492a46d419478daf65eeaed5036cfcfa3e5\n" }, { "commit": "b8a1d90d4ebbf982fe7e1c6303c4b8a8bb87fddb", "tree": "d958fcf07af32296894150cb37ba1776776601fa", "parents": [ "62c87df826a02f4ecf20c18bc4b52144c3b50724" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Mon May 17 08:57:43 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 13:34:27 2021 -0700" }, "message": "[dexter] Remove unnecessary double check on conditional breakpoints\n\nRemove the `ConditionalController._conditional_met` method. This was missed in\nthe recent ConditionalController refactor (D98699). We don\u0027t need to check that\nthe conditions for a conditional breakpoint have been met because\n`DebuggerBase.get_triggered_breakpoint_ids` returns the set of ids for\nbreakpoints which have been triggered.\n\nTo get the \"triggered breakpoints\" from lldb we use `GetStopReasonDataCount`\nand `GetStopReasonDataAtIndex`. It seems that these functions count all\nbreakpoints associated with the location which lldb has stopped at, regardless\nof their condition. i.e. Even if we have two breakpoints at the same source\nlocation that have mutually exclusive conditions, both will be found this way\nwhen either condition is true. To get around this, we store a map of breakpoint\n{id: condition} `_breakpoint_conditions` and evaluate the conditions of the\ntriggered breakpoints to filter the set down to those which are unconditional\nor have a condition which evaluates to true.\n\nEssentially we are just moving the condition double check from a general\ndebugger controller into the lldb specific wrapper. This tidy up will help make\nupcoming patches simpler.\n\nReviewed By: chrisjackson\n\nDifferential Revision: https://reviews.llvm.org/D101431\n\nGitOrigin-RevId: a11117a7c0a23f37bdee5c611bbaac22e89c775d\n" }, { "commit": "62c87df826a02f4ecf20c18bc4b52144c3b50724", "tree": "33156d6a94a86d9ad270eed74f4dc88c81d757ba", "parents": [ "c5dba7492894cce23d5b7e356d0dd7095491082c" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Mon Apr 26 16:33:10 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun May 09 12:01:57 2021 -0700" }, "message": "[dexter] Update failing regression test\n\nCommit 7deb970ef added REQUIRES and UNSUPPORTED lines to a test which is\nsensitive to line number changes. Update the line number reference in the test.\n\nGitOrigin-RevId: 40df6bac9c5c93f96181c51bbea99af5b1c5fc53\n" }, { "commit": "c5dba7492894cce23d5b7e356d0dd7095491082c", "tree": "7e0ea1eca8d4e2124a2bbfad6a2bf8619d708f07", "parents": [ "fbbfe4836c0801ebc1b795231d908a8e0ea0c8ec" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Fri Apr 23 17:38:44 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun May 09 12:01:54 2021 -0700" }, "message": "Drop a REQUIRES: lldb on a dexter regression test\n\nAs this is a test that actually gets to operating the debugger, it\nneeds to be limited to scenarios where the debugger is available.\n\n(We\u0027ll file this in the set of things Dexter doesn\u0027t handle gracefully..)\n\nGitOrigin-RevId: 7deb970efb0f1a7ccb04740710f683e00e005eae\n" }, { "commit": "fbbfe4836c0801ebc1b795231d908a8e0ea0c8ec", "tree": "e564b4bb08a07946c332835427ea4f26fdd8589d", "parents": [ "bf5e28a79bd71648f157f6331a5b095cec4cb46e" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Fri Apr 23 12:00:16 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun May 09 12:01:50 2021 -0700" }, "message": "[dexter] Add keyword argument \u0027on_line\u0027 to DexLabel\n\nAdd optional keyword argument \u0027on_line\u0027 to DexLabel to label the specifed line\ninstead of the line the command is found on.\n\nThis will be helpful when used alongside DexDeclareFile (D99651).\n\nReviewed By: TWeaver\n\nDifferential Revision: https://reviews.llvm.org/D101055\n\nGitOrigin-RevId: 487ab5345920b2cfd2162f04b467dd39fb1a8627\n" }, { "commit": "bf5e28a79bd71648f157f6331a5b095cec4cb46e", "tree": "3cfdc68835fa5825e81a82a2c8afa3ecce6c0057", "parents": [ "e537439771d5b783d4df5a16e9c12ad664238b09" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Tue Mar 23 11:12:59 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Mar 25 13:24:56 2021 -0700" }, "message": "[dexter] Fix DexLimitSteps when breakpoint can\u0027t be set at requested location\n\nUsing a DexLimitSteps command forces dexter to use the ConditionalController\ndebugger controller. At each breakpoint the ConditionalController needs to\nunderstand which one has been hit. Prior to this patch, upon hitting a\nbreakpoint, dexter used the current source location to look up which requested\nbreakpoint had been hit.\n\nA breakpoint may not get set at the exact location that the user (dexter)\nrequests. For example, if the requested breakpoint location doesn\u0027t exist\nin the line table then then debuggers will (usually, AFAICT) set the breakpoint\nat the next available valid breakpoint location.\n\nThis meant that, occasionally in unoptimised programs and frequently in\noptimised programs, the ConditionalController was failing to determine which\nbreakpoint had been hit.\n\nThis is the fix:\n\nChange the DebuggerBase breakpoint interface to use opaque breakpoint ids\ninstead of using source location to identify breakpoints, and update the\nConditionalController to track breakpoints instead of locations.\n\nThese now return a breakpoint id:\n\n add_breakpoint(self, file_, line)\n _add_breakpoint(self, file_, line)\n add_conditional_breakpoint(self, file_, line, condition)\n _add_conditional_breakpoint(self, file_, line, condition)\n\nReplace:\n\n delete_conditional_breakpoint(self, file_, line, condition)\n _delete_conditional_breakpoint(self, file_, line, condition)\n\nwith:\n\n delete_breakpoint(self, id)\n\nAdd:\n\n get_triggered_breakpoint_ids(self)\n\nA breakpoint id is guaranteed to be unique for each requested breakpoint, even\nfor duplicate breakpoint requests. Identifying breakpoints like this, instead\nof by location, removes the possibility of mixing up requested and bound\nbreakpoints.\n\nThis closely matches the LLDB debugger interface so little work was required in\nLLDB.py, but some extra bookkeeping is required in VisualStudio.py to maintain\nthe new breakpoint id semantics. No implementation work has been done in\ndbgeng.py as DexLimitSteps doesn\u0027t seem to support dbgeng at the moment.\n\nTesting\nAdded:\ndexter/feature_tests/commands/perfect/limit_steps/limit_steps_line_mismatch.cpp\n\nThere were no unexpected failures running the full debuginfo-tests suite.\n\nThe regression tests use dbgeng on windows by default, and as mentioned above\ndbgeng isn\u0027t supported yet, so I have also manually tested (i.e. without lit)\nthat this specific test works as expected with clang and Visual Studio 2017 on\nWindows.\n\nReviewed By: TWeaver\n\nDifferential Revision: https://reviews.llvm.org/D98699\n\nGitOrigin-RevId: faf5f1cbbac020c7a6c6de188ae96a4dc15b5cdd\n" }, { "commit": "e537439771d5b783d4df5a16e9c12ad664238b09", "tree": "e9f90f5742c4905ec84b951fbcfeb1c3128ca4f8", "parents": [ "ad5017b5fedb66f6b41a5cdce0a36f382cba77c6" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Mon Mar 15 11:28:28 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Mar 18 00:16:18 2021 -0700" }, "message": "[dexter] Check path !\u003d None before calling os.path.exists\n\nThe test optnone-simple-functions.cpp added in D97668 fails on macOS.\nos.path.exists raises an exception because we pass it None. Guard against this.\n\nRelated revision: https://reviews.llvm.org/D97668\n\nGitOrigin-RevId: 61d314024dc447e49481cb8494bf7165f9ec1323\n" }, { "commit": "ad5017b5fedb66f6b41a5cdce0a36f382cba77c6", "tree": "a02184601a11e4ec5d30534050121ef8a76eb26e", "parents": [ "e76a29066b6dea48bf303a111d029e15709eb3ef" ], "author": { "name": "Nathan Chancellor", "email": "nathan@kernel.org", "time": "Mon Mar 15 05:09:52 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Mar 18 00:16:15 2021 -0700" }, "message": "debuginfo-tests: Fix check-gdb-mlir-support build after MLIR API change in a4bb667d831c\n\nReviewed By: mehdi_amini\n\nDifferential Revision: https://reviews.llvm.org/D98613\n\nGitOrigin-RevId: 633549f73eca2e8cd26b01e980bace833713f217\n" }, { "commit": "e76a29066b6dea48bf303a111d029e15709eb3ef", "tree": "ebdebbff2b15b59271da931b93fe3a7cde3cbeb9", "parents": [ "c65379df5aaee14ea69d3d8bc5be63a249925641" ], "author": { "name": "Mehdi Amini", "email": "joker.eph@gmail.com", "time": "Tue Mar 02 19:20:10 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Mar 18 00:16:12 2021 -0700" }, "message": "Fix `debuginfo-tests/llvm-prettyprinters` build after MLIR API change in e6260ad043d8 (NFC)\n\nGitOrigin-RevId: 068aa12f96afdb0aa31e2986fa23684f41197cd3\n" }, { "commit": "c65379df5aaee14ea69d3d8bc5be63a249925641", "tree": "e6bb844997d12c0e121e6fb9f0097f4a6817ca0d", "parents": [ "cc194fc21c4fa8761fb8da9e2b52799216d7272c" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Tue Mar 02 08:45:53 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Mar 18 00:16:09 2021 -0700" }, "message": "[debuginfo-tests] Add some optnone tests\n\nAdd dexter tests using the optnone attribute in various scenarios. Our users\nhave found optnone useful when debugging optimised code. We have these tests\ndownstream (and one upstream already: D89873) and we would like to contribute\nthem if there is any interest.\n\nThe tests are fairly self explanatory. Testing optnone with:\n * optnone-fastmath.cpp: floats and -ffast-math,\n * optnone-simple-functions: simple functions and integer arithmetic,\n * optnone-struct-and-methods: a struct with methods,\n * optnone-vectors-and-functions: templates and integer vector arithmetic.\n\noptnone-vectors-and-functions contains two FIXMEs. The first problem is that\nlldb seems to struggle with evaluating expressions with the templates used\nhere (example below). Perhaps this is PR42920?\n\n (lldb) p TypeTraits\u003cint __attribute__((ext_vector_type(4)))\u003e::NumElements\n error: \u003cuser expression 0\u003e:1:1: no template named \u0027TypeTraits\u0027\n TypeTraits\u003cint __attribute__((ext_vector_type(4)))\u003e::NumElements\n ^\nThe second is that while lldb cannot evaluate the following expression, gdb\ncan, but it reports that the variable has been optimzed away. It does this when\ncompiling at O0 too. llvm-dwarfdump shows that MysteryNumber does have a\nlocation. I don\u0027t know whether the DIE is bad or if both debuggers just don\u0027t\nsupport it.\n\n TypeTraits\u003cint __attribute__((ext_vector_type(4)))\u003e::MysteryNumber\n\n DW_TAG_variable\n DW_AT_specification (0x0000006b \"MysteryNumber\")\n DW_AT_location (DW_OP_addr 0x601028)\n DW_AT_linkage_name (\"_ZN10TypeTraitsIDv4_iE13MysteryNumberE\")\n\nReviewed By: rnk\n\nDifferential Revision: https://reviews.llvm.org/D97668\n\nGitOrigin-RevId: 14be3f0e8848ee5a2dc215404c0391de4dc9e252\n" }, { "commit": "cc194fc21c4fa8761fb8da9e2b52799216d7272c", "tree": "38b0c5093131da15764c605c5769ca2d0b6876f7", "parents": [ "7c4d9725da2b3edf6de377868c29947a7f3c2219" ], "author": { "name": "Harmen Stoppels", "email": "harmenstoppels@gmail.com", "time": "Thu Feb 25 11:31:42 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 26 21:55:43 2021 -0800" }, "message": "Prefer /usr/bin/env xxx over /usr/bin/xxx where xxx \u003d perl, python, awk\n\nAllow users to use a non-system version of perl, python and awk, which is useful\nin certain package managers.\n\nReviewed By: JDevlieghere, MaskRay\n\nDifferential Revision: https://reviews.llvm.org/D95119\n\nGitOrigin-RevId: a54f160b3a98b91cd241a555d904a6b6453affc4\n" }, { "commit": "7c4d9725da2b3edf6de377868c29947a7f3c2219", "tree": "c0bcc718c1f63546e5cb39a41cfa9804d5ba7766", "parents": [ "31fc0156132098db0d5844433ad1d606825a77f2" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Wed Feb 24 11:09:18 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 26 21:55:39 2021 -0800" }, "message": "[dexter] Force dexter tests to use the host triple\n\nIf the default target and host triple don\u0027t match then print the following\nmessage when the lit test runs:\n\n Forcing dexter tests to use host triple {HOST_TRIPLE}.\n\nIf we can\u0027t target the host arch then, when lit runs, we mark\nthe dexter test directories as UNSUPPORTED and print the message:\n\n Host triple {HOST_TRIPLE} not supported. Skipping dexter tests in\n the debuginfo-tests project.\n\nReviewed By: rnk\n\nDifferential Revision: https://reviews.llvm.org/D96494\n\nGitOrigin-RevId: 52bc1c1bd5397ef6ff37214a0186a47aa3348b72\n" }, { "commit": "31fc0156132098db0d5844433ad1d606825a77f2", "tree": "691db91fc1d6992e8eb025e02dd507b8949b17d2", "parents": [ "504cb77e26536b31ae9091a015dde93d19629624" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Fri Feb 19 08:44:32 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 26 21:55:36 2021 -0800" }, "message": "[debuginfo-tests] Recommit test sret.cpp\n\nThis test was accidently removed when the directory structure was shuffled\naround for dexter in f78c236efda8.\n\nReviewed By: aprantl\n\nDifferential Revision: https://reviews.llvm.org/D96968\n\nGitOrigin-RevId: 54b6d01685ef3b3bce4f0b1e9262291a9d535725\n" }, { "commit": "504cb77e26536b31ae9091a015dde93d19629624", "tree": "bd13c488b65e0bbef711e6a233a5a3452d78a4fa", "parents": [ "3209a4d1686a8c453ad4e7c6c0ea3fd056e18113" ], "author": { "name": "James Henderson", "email": "james.henderson@sony.com", "time": "Tue Feb 09 14:24:18 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 19 01:01:13 2021 -0800" }, "message": "[debuginfo-tests] Delete unused/duplicate imports\n\nDifferential Revision: https://reviews.llvm.org/D96502\n\nReviewed by: aprantl\n\nGitOrigin-RevId: e8b9da712f579cbd18f5b993e984c18c501f4802\n" }, { "commit": "3209a4d1686a8c453ad4e7c6c0ea3fd056e18113", "tree": "14ee755ac1f02856b1c543de556ebf2f23bebc4e", "parents": [ "719be35916d2c729f24873ad1701bcf0f759ac58" ], "author": { "name": "James Henderson", "email": "james.henderson@sony.com", "time": "Tue Feb 09 11:12:49 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 19 01:01:10 2021 -0800" }, "message": "[debuginfo-tests] Remove some unused config variables\n\nDifferential Revision: https://reviews.llvm.org/D96500\n\nReviewed by: aprantl\n\nGitOrigin-RevId: d6236524993e81904e75e9ddc3d497230b858a89\n" }, { "commit": "719be35916d2c729f24873ad1701bcf0f759ac58", "tree": "ae954dcc5fe1d8225e8dd0c6e082f505fc776eae", "parents": [ "2787bff433cb6001292a106e9abb5eea774318be" ], "author": { "name": "James Henderson", "email": "james.henderson@sony.com", "time": "Mon Feb 08 15:46:40 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 19 01:01:07 2021 -0800" }, "message": "[debuginfo-tests] Remove explicit checks for Python 3\n\nLLVM has a minimum requirement of python 3.6 now, and Python is\nexplicitly checked for in the LLVM CMakeLists.txt, so this check is no\nlonger needed here.\n\nDifferential Revision: https://reviews.llvm.org/D96499\n\nReviewed by: aprantl\n\nGitOrigin-RevId: 6c330f0df8da7f6b94f511fc3f5573003fdda093\n" }, { "commit": "2787bff433cb6001292a106e9abb5eea774318be", "tree": "489564ebd869496cf9adf1e1f08c844488e465aa", "parents": [ "c02e514ce5d20b9236033e25a66a4a34e512f050" ], "author": { "name": "James Henderson", "email": "james.henderson@sony.com", "time": "Wed Feb 10 15:14:32 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 19 01:01:04 2021 -0800" }, "message": "[test][Dexter] Fix test failure if space in python path\n\nThe \u0027%dexter_regression_test\u0027 substitution was missing quotes around the\npython executable, unlike other substitutions of a similar nature in the\nfile. This changes fixes the issue.\n\nDifferential Revision: https://reviews.llvm.org/D96420\n\nReviewed by: jmorse, aganea\n\nGitOrigin-RevId: a31eae840525e9292a3a42c1fdac3fc594f42949\n" }, { "commit": "c02e514ce5d20b9236033e25a66a4a34e512f050", "tree": "9b58fbb974fef6e39510e48043cf0031937e4d4a", "parents": [ "05da80d02f520043b632f26d6aaf1de1ab838f73" ], "author": { "name": "Raphael Isemann", "email": "teemperor@gmail.com", "time": "Thu Feb 11 10:31:15 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 19 01:01:00 2021 -0800" }, "message": "[DebugInfo] Prevent inlining in NRVO-string test cases\n\nSince the new pass manager has been enabled by default these tests had their\n-O1 variations failing due to the tested functions being inlined. This just\nadds no_inline to the respective code similar to what we did in other\ntests (e.g. aa56b30014d96700b2fc945be11866789f341d38 ).\n\nGitOrigin-RevId: a874d182c61c7f8e5291c718b40be0e133bf21f0\n" }, { "commit": "05da80d02f520043b632f26d6aaf1de1ab838f73", "tree": "df8ee624b13e32c30f1b7807583066598df891cb", "parents": [ "029d91267427da3063182e94e3df7807126e641c" ], "author": { "name": "Tres Popp", "email": "tpopp@google.com", "time": "Mon Feb 08 09:44:03 2021 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Feb 08 14:02:32 2021 -0800" }, "message": "Revert \"Revert \"Reorder MLIRContext location in BuiltinAttributes.h\"\"\n\nThis reverts commit 511dd4f4383b1c2873beac4dbea2df302f1f9d0c along with\na couple fixes.\n\nOriginal message:\nNow the context is the first, rather than the last input.\n\nThis better matches the rest of the infrastructure and makes\nit easier to move these types to being declaratively specified.\n\nPhabricator: https://reviews.llvm.org/D96111\nGitOrigin-RevId: c2c83e97c3ac98ddf5bd685cbfba3f620f59fa51\n" }, { "commit": "029d91267427da3063182e94e3df7807126e641c", "tree": "6614e7064662f43dc1781609f6ed0ade0206e9dc", "parents": [ "19d93713cab4aaa0748173b2e40ed0218cb3180d" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Thu Feb 04 14:54:12 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 05 01:21:54 2021 -0800" }, "message": "[Dexter] Avoid infinite loop in dbgeng driver\n\nThis method of the dbgeng debugger driver used to just \"pass\", as setting\ndbgeng free running still leads to numerous errors. This wasn\u0027t a problem\nin the past because, as it turns out, nothing called the go method.\nHowever, a recent refactor uses it.\n\nRather than launch dbgeng free running, instead have it single step one\nstep forwards. This is slow, but it makes progress, where previously we\nweren\u0027t.\n\nDifferential Revision: https://reviews.llvm.org/D91737\n\nGitOrigin-RevId: c8e6cde95c8c14941b87f33962e89873d0638c7e\n" }, { "commit": "19d93713cab4aaa0748173b2e40ed0218cb3180d", "tree": "2737c80848c1c1ad0ab893aaead01410784e2fbb", "parents": [ "189c9738f8caecfbe2154bada8b082ff566a3bfe" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Tue Jan 19 12:31:55 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 19 15:49:07 2021 -0800" }, "message": "[DebugInfo][dexter] Tweak dexter test for merged values\n\nTweak dexter-tests/memvars/inline-escaping-function.c added in D94761\n(b7e516202eb6) by adding a \u0027param\u0027 use after the merge point. The test XFAILS\nwith and without this change, but without it the test looks very similar to\nmemvars/unused-merged-value.c. The test now demonstrates the problem more\nclearly.\n\nGitOrigin-RevId: d77a57208770470d546194ad81bd01ae5d94df0d\n" }, { "commit": "189c9738f8caecfbe2154bada8b082ff566a3bfe", "tree": "f640fc1b36b34dad217388d1bd5353e268412b01", "parents": [ "b0930f044521f001e4c67b4c45d164d3885ea26c" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Tue Jan 19 11:04:25 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 19 15:49:04 2021 -0800" }, "message": "[DebugInfo][dexter] Add dexter tests for merged values\n\nThese dexter tests illustrate PR48719, the summary of which is:\n\nSometimes we insert dbg.values for merged values (PHIs) when promoting\nvariables, sometimes we don\u0027t. Sometimes there is no PHI because the merged\nvalue is never used. It doesn\u0027t matter because LiveDebugValues understands these\nmerged values (implicit or otherwise) and correctly updates the debug\ninfo. Importantly, these merged variable values (which may or may not exist as\nPHIs, and may or not be represented with dbg.values) are //always// implicitly\ndefined by the combination of incoming edges and the incoming variable locations\nalong those edges by virtue of LiveDebugValues existing. Unfortunately, it is\npossible to mess with the CFG and remove / move these edges before\nLiveDebugValues runs. In this case our debug info model only works when the\nmerged value is tracked by a dbg.value. Currently, this is only done rigorously\nfor variables which are A) promoted in the first round of mem2reg and B) are\nused after the merge point.\n\nAs an example, compile the following source with -O3 -g and step through with a\ndebugger. You will see parama\u003d5 throughout the function fun which is incorrect -\nwe expect to see param\u003d20 after the conditional assignment.\n\n __attribute__((optnone))\n void esc(int* p) {}\n\n __attribute__((optnone))\n void fluff() {}\n\n __attribute__((noinline))\n int fun(int parama, int paramb) {\n if (parama)\n parama \u003d paramb;\n fluff(); // DexLabel(\u0027s0\u0027)\n esc(\u0026parama);\n return 0;\n }\n\n int main() {\n return fun(5, 20);\n }\n\n1. parama is escaped by esc(\u0026parama) so it is not promoted by\n SROA/mem2reg (failing condition \"A\" above).\n2. InstCombine\u0027s LowerDbgDeclare converts the dbg.declare to a set of\n dbg.values (tracking the stored SSA values).\n3. InstCombine replaces the two stores to parama\u0027s alloca (the initial\n parameter register store in entry and the assignment in if.then) with a\n PHI+store in the common sucessor.\n4. SimplifyCFG folds the blocks together and converts the PHI to a\n select.\n\nThe debug info is not updated to account for the merged value in the successor\nprior to SimplifyCFG when it exists as a PHI, or during when it becomes a\nselect.\n\nAs with D89543, which added some dexter tests for escaped locals, the idea is\nto build a set of source-level tests which highlights existing issues and\nmight be useful in evaluating a new debug info model.\n\nReviewed By: rnk\n\nDifferential Revision: https://reviews.llvm.org/D94761\n\nGitOrigin-RevId: b7e516202eb66e004c49b89964bd5b30b287af87\n" }, { "commit": "b0930f044521f001e4c67b4c45d164d3885ea26c", "tree": "ac89099f1fb60b9dd98b3101162d2680a9340029", "parents": [ "1329fff414b7fcac3d3610f4f552394de20ace95" ], "author": { "name": "Mehdi Amini", "email": "joker.eph@gmail.com", "time": "Thu Jan 07 21:30:39 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 07 14:05:35 2021 -0800" }, "message": "Fix check-gdb-mlir-support build after MLIR API changed to take Context as first argument\n\nGitOrigin-RevId: 9e1aaa9943b814c22ae03f4abb3171dac8062801\n" }, { "commit": "1329fff414b7fcac3d3610f4f552394de20ace95", "tree": "cf8f911c768a80c51d8fa74fe38c066b60452f50", "parents": [ "77e3b544d909c243ea946bd341e371d8c113e49b" ], "author": { "name": "Mehdi Amini", "email": "joker.eph@gmail.com", "time": "Thu Jan 07 21:25:59 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 07 14:05:32 2021 -0800" }, "message": "Fix include path for check-gdb-mlir-support to include the MLIR binary dir\n\nThis fixes a build failure:\n\nfatal error: \u0027mlir/IR/BuiltinTypes.h.inc\u0027 file not found\n\nGitOrigin-RevId: 476db17dcb64ef3ec6e247f4b1c673b57f61a367\n" }, { "commit": "77e3b544d909c243ea946bd341e371d8c113e49b", "tree": "011f0cff2ae5bfbf5fea69dc5abaa381f3c7bff6", "parents": [ "b8e4f7fabe2d88b83d6f880136492e8a8070494a" ], "author": { "name": "Fangrui Song", "email": "i@maskray.me", "time": "Wed Jan 06 20:39:07 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 07 14:05:29 2021 -0800" }, "message": "[debuginfo-test] Fix -Wunused-value\n\nGitOrigin-RevId: 25bf4a8f4285865c11566d04cca19562de5e622f\n" }, { "commit": "b8e4f7fabe2d88b83d6f880136492e8a8070494a", "tree": "c55217d746dd2e6b2f25e7d4162e82f31ecbbd16", "parents": [ "67e4c39e3e7a873439451ef165080c82f803e243" ], "author": { "name": "Krzysztof Parzyszek", "email": "kparzysz@quicinc.com", "time": "Fri Dec 04 15:54:29 2020 -0600" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Dec 08 16:47:05 2020 -0800" }, "message": "Include BuiltinAttributes.h in llvm-prettyprinters/gdb/mlir-support.cpp\n\nThis header was introduced in c7cae0e4fa4e1ed4bdca186096a408578225fc2b.\n\nGitOrigin-RevId: 1b404ad51e93724ecd74c66f1a11f03568398c57\n" }, { "commit": "67e4c39e3e7a873439451ef165080c82f803e243", "tree": "81d209cbc5bf8ebcba4e5b78352fe58988adb542", "parents": [ "99ef19c0e207c4716c19651ce97a58356d06b7ec" ], "author": { "name": "River Riddle", "email": "riddleriver@gmail.com", "time": "Thu Dec 03 17:22:29 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Dec 08 16:47:01 2020 -0800" }, "message": "[mlir][Types][NFC] Move all of the builtin Type classes to BuiltinTypes.h\n\nThis is part of a larger refactoring the better congregates the builtin structures under the BuiltinDialect. This also removes the problematic \"standard\" naming that clashes with the \"standard\" dialect, which is not defined within IR/. A temporary forward is placed in StandardTypes.h to allow time for downstream users to replaced references.\n\nDifferential Revision: https://reviews.llvm.org/D92435\n\nGitOrigin-RevId: 09f7a55fad924b894fadef0f0edccffdfda7838a\n" }, { "commit": "99ef19c0e207c4716c19651ce97a58356d06b7ec", "tree": "b89f832a9284eec4ee91a8f6d84b4b86f94355cd", "parents": [ "fab8cce6f46f45820268bf3f899b0081c3c2810a" ], "author": { "name": "Moritz Sichert", "email": "sichert@in.tum.de", "time": "Wed Nov 18 15:36:06 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Nov 30 01:40:35 2020 -0800" }, "message": "Added GDB pretty printer for StringMap\n\nReviewed By: csigg, dblaikie\n\nDifferential Revision: https://reviews.llvm.org/D91183\n\nGitOrigin-RevId: 5747380772e0bbf5762e5ed9c39e6c9265bafc87\n" }, { "commit": "fab8cce6f46f45820268bf3f899b0081c3c2810a", "tree": "6c4c3165f6c1c216229ff9a76b2da79afcef62f9", "parents": [ "e53d375ec6f7f279996dab9940a35287896678f0" ], "author": { "name": "Jonas Devlieghere", "email": "jonas@devlieghere.com", "time": "Fri Nov 06 13:04:55 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Nov 10 14:02:53 2020 -0800" }, "message": "[debuginfo-tests] Skip optnone-loops.cpp on Darwin\n\nThis is failing on GreenDragon:\nhttp://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/24745/\n\nGitOrigin-RevId: 2993850237b089d134cec2776312b61ef9bff757\n" }, { "commit": "e53d375ec6f7f279996dab9940a35287896678f0", "tree": "e503757a1c68d16e2e5ab3e508bcb94afcf649df", "parents": [ "06ba55e96224f7d8087410459c19cd9c31806537" ], "author": { "name": "Louis Dionne", "email": "ldionne@apple.com", "time": "Fri Nov 06 08:17:02 2020 -0500" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Nov 10 14:02:52 2020 -0800" }, "message": "[debuginfo-tests] NFC: Move test that was committed to the wrong location\n\nDifferential Revision: https://reviews.llvm.org/D90931\n\nGitOrigin-RevId: 08e6903eb4df9db2a88fd1f189c41d1a42908768\n" }, { "commit": "06ba55e96224f7d8087410459c19cd9c31806537", "tree": "c78dc87bd2dd0ce0a07b3d6664f6a1ca34000d5d", "parents": [ "76de5f519f38af7e16f082e630e2c98a23208551" ], "author": { "name": "Nabeel Omer", "email": "nabeel.omer@sony.com", "time": "Wed Nov 04 16:57:19 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Nov 10 14:02:48 2020 -0800" }, "message": "[Dexter] add visual studio 2019 debugger support\n\nAdds visual studio debugger support to dexter via option --debugger vs2019\n\nDifferential Revision: https://reviews.llvm.org/D89803\n\nAuthor: Nabeel Omer \u003cnabeel.omer@sony.com\u003e\nGitOrigin-RevId: 3ebcef4b737746bd8c60ff77b53f4fc79ac4b4d8\n" }, { "commit": "76de5f519f38af7e16f082e630e2c98a23208551", "tree": "72d74a84793ffc164d74217dc6828aa4cd4b4910", "parents": [ "6e959db299bcb3f1fd01699688807badbc9d1edf" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Wed Oct 28 17:33:29 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 17:40:17 2020 -0700" }, "message": "[debuginfo-tests][dexter] add requires lldb to two tests\n\nboth deferred_globals.cpp namespace.cpp require lldb in order to run and will\nfail if it\u0027s not available.\n\nadd the required lines to the top of the tests.\n\nGitOrigin-RevId: f2f4554f88040954ec0059764ef412a97ce5f19a\n" }, { "commit": "6e959db299bcb3f1fd01699688807badbc9d1edf", "tree": "d94b9f32932271f1a3c23ea81123a6926f7a29c5", "parents": [ "6a852443c60c1ad8fc1aee446a0c82adb2fd60c7" ], "author": { "name": "Nabeel Omer", "email": "nabeel.omer@sony.com", "time": "Wed Oct 28 14:21:40 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 17:40:15 2020 -0700" }, "message": "[debuginfo-tests][dexter] Add two new debug experience tests\n\ndeferred_globals.cpp: Verify that debug information for a local variable does\nnot hide a global definition that has the same name\n\nnamespace.cpp: Ensure that the debug information for a global variable\nincludes namespace information.\n\nDifferential Revision: https://reviews.llvm.org/D89462\n\nAuthor: Nabeel Omer \u003cnabeel.omer@sony.com\u003e\nGitOrigin-RevId: afc44efc262b27e8d6cdcf86d40a820ecd450520\n" }, { "commit": "6a852443c60c1ad8fc1aee446a0c82adb2fd60c7", "tree": "4a8d74accf9a67bab3da98729af214eb8469d01c", "parents": [ "ade09b1190b89f7a9b85b2d757269ea1f616af87" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Mon Oct 26 10:20:50 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 17:40:14 2020 -0700" }, "message": "[DebugInfo][dexter] Add dexter tests for escaped locals\n\nRecently there has been renewed interest in improving debug-info for variables\nthat (partially or otherwise) live on the stack in optimised code.\n\nAt the moment instcombine speculates that stack slots are probably going to be\npromoted to registers, and prepares the debug-info accordingly. It runs a\nfunction called LowerDbgDeclare which converts dbg.declares to a set of\ndbg.values after loads, and before stores and calls. Sometimes the stack\nlocation remains (e.g. for escaped locals). If any dbg.values become undef\nwhere the stack location is still valid we end up unnecessarily reducing\nvariable location coverage due to our inability to track multiple locations\nsimultaneously. There is a flag to disable this feature\n(-instcombine-lower-dbg-declare\u003d0), which prevents this conversion at the cost\nof sometimes providing incorrect location info in the face of DSE, DCE, GVN,\nCSE etc.\n\nThis has been discussed fairly extensively on PR34136.\n\nThe idea of these tests is to provide examples of situations that we should\nconsider when designing a new system, to aid discussions and eventually help\nevaluate the implementation.\n\nDexter isn\u0027t ideal for observing specific optimisation behaviour. Writing an\nexaustive test suite would be difficult, and the resultant suite would be\nfragile. However, I think having some concrete executable examples is useful\nat least as a reference.\n\nDifferential Revision: https://reviews.llvm.org/D89543\n\nGitOrigin-RevId: 66d03af88c2e4932ae53787e07f502b90ab220cc\n" }, { "commit": "ade09b1190b89f7a9b85b2d757269ea1f616af87", "tree": "63c5839afc97ba48224a5dbd35d5e8d28f973c8d", "parents": [ "75a62a544e75e23732d538eafbce91b39ac2fe84" ], "author": { "name": "Nabeel Omer", "email": "nabeel.omer@sony.com", "time": "Mon Oct 19 16:38:49 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 17:40:13 2020 -0700" }, "message": "[Dexter][NFC] Add Missing Commands to Commands.md Contents\n\n NFC patch simply updates the commands.md documentation contents with missing\n links to the DexLimitSteps and DexLabel command documentation.\n\n Differential Revision: https://reviews.llvm.org/D89689\n\n Author: Nabeel Omer \u003cnabeel.omer@sony.com\u003e\n\nGitOrigin-RevId: 7d8c19a4e9ed8f39eda0f0f383d5ff8f1485471b\n" }, { "commit": "75a62a544e75e23732d538eafbce91b39ac2fe84", "tree": "e8f1d1616c391be4b60f06c06f8d0e7914f49522", "parents": [ "cc191da21c53c0ff9ac5c243b499097623d8aa32" ], "author": { "name": "Christian Sigg", "email": "csigg@google.com", "time": "Tue Sep 29 15:19:54 2020 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 17:40:12 2020 -0700" }, "message": "Add GDB prettyprinters for a few more MLIR types.\n\nReviewed By: dblaikie, jpienaar\n\nDifferential Revision: https://reviews.llvm.org/D87159\n\nGitOrigin-RevId: e9b38841619f20a6f4c8657880fd487083ba499a\n" }, { "commit": "cc191da21c53c0ff9ac5c243b499097623d8aa32", "tree": "b82ab3eaad61afbc130a28d4fe55aca140359552", "parents": [ "c5835a69a02f14dfea0613025d8ffe113a346a2a" ], "author": { "name": "OCHyams", "email": "orlando.hyams@sony.com", "time": "Thu Sep 03 08:28:15 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 17:40:11 2020 -0700" }, "message": "Revert \"Adding GDB PrettyPrinter for mlir::Identifier.\"\n\nThis reverts commit 9e9e6e698d8ef5dc5b7576058f2022aab2534a52.\n\nThis commit is causing builds that include the \u0027debuginfo-tests\u0027 project to\nfail.\n\nApple has a public bot which shows the failure:\nhttp://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/23667/console\n\nGitOrigin-RevId: 485e6db8729383345da9796218d99843449057ac\n" }, { "commit": "c5835a69a02f14dfea0613025d8ffe113a346a2a", "tree": "949d6053ee2779e161da661cc677dd12aba8cd11", "parents": [ "d22d4ab711242f3522c0c06cd68d5adecf54ef50" ], "author": { "name": "Christian Sigg", "email": "csigg@google.com", "time": "Tue Sep 01 15:43:38 2020 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 17:40:09 2020 -0700" }, "message": "Adding GDB PrettyPrinter for mlir::Identifier.\n\nThis is the first bit from D73546. Primarily setting up the corresponding test. Will add more pretty printers in a separate revision.\n\nReviewed By: dblaikie\n\nDifferential Revision: https://reviews.llvm.org/D86937\n\nGitOrigin-RevId: 9e9e6e698d8ef5dc5b7576058f2022aab2534a52\n" }, { "commit": "d22d4ab711242f3522c0c06cd68d5adecf54ef50", "tree": "b82ab3eaad61afbc130a28d4fe55aca140359552", "parents": [ "60b23469f7988f18bde96cb28bb704aa639722c7" ], "author": { "name": "serge-sans-paille", "email": "sguelton@redhat.com", "time": "Wed Jul 15 09:31:13 2020 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:19 2020 -0700" }, "message": "Harmonize Python shebang\n\nDifferential Revision: https://reviews.llvm.org/D83857\n\nGitOrigin-RevId: 515bc8c1554f03515565878ea3d35cffdc6fd195\n" }, { "commit": "60b23469f7988f18bde96cb28bb704aa639722c7", "tree": "cbc44680b9fb1f599f0e9bd3b01abb59799605b9", "parents": [ "ab6d6ba2e738aee51902a1438f45673de573389e" ], "author": { "name": "Fangrui Song", "email": "maskray@google.com", "time": "Fri Jun 26 11:18:18 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:17 2020 -0700" }, "message": "[CMake] Add check-debuginfo-* targets\n\n* check-debuginfo-dexter runs lit tests under debuginfo-tests/dexter/\n* check-debuginfo-llgdb-tests runs lit tests under debuginfo-tests/llgdb-tests/\n* ...\n\nReviewed By: tbosch\n\nDifferential Revision: https://reviews.llvm.org/D82605\n\nGitOrigin-RevId: ac567eec119f7d288c6f47921b348aeea7d743cd\n" }, { "commit": "ab6d6ba2e738aee51902a1438f45673de573389e", "tree": "b6d775ddd5476981c927e6726bb0e46906d96793", "parents": [ "ddaef7a2619ee5a18da8b4b7822e64a3ddbf683b" ], "author": { "name": "Fangrui Song", "email": "maskray@google.com", "time": "Thu Jun 25 18:28:12 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:16 2020 -0700" }, "message": "[CMake] Add optional lldb dependency to DEBUGINFO_TEST_DEPS\n\nif(TARGET asan) is still brittle as it depends on the order of\ncompiler-rt and debuginfo-tests in LLVM_ENABLE_PROJECTS.\n\nGitOrigin-RevId: 161ae1f39816edf667aaa190bce702a86879c7bd\n" }, { "commit": "ddaef7a2619ee5a18da8b4b7822e64a3ddbf683b", "tree": "0d1bcedb7b6873dd0147ac6ff1accba05c23ab6a", "parents": [ "4cdccc5108610574534e7538cffac92f20b38dc7" ], "author": { "name": "Fangrui Song", "email": "maskray@google.com", "time": "Thu Jun 25 17:56:22 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:12 2020 -0700" }, "message": "[CMake] Add optional asan \u0026 safestack dependencies to DEBUGINFO_TEST_DEPS\n\nGitOrigin-RevId: e477a5f6c88de0ef7745694ec5f00bd69da23177\n" }, { "commit": "4cdccc5108610574534e7538cffac92f20b38dc7", "tree": "05c0118031fca71f90b19c1f2b08bd45d39d188a", "parents": [ "4dda378488eaa92026d3735286a243b2e859b00b" ], "author": { "name": "Fangrui Song", "email": "maskray@google.com", "time": "Thu Jun 25 17:35:01 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:08 2020 -0700" }, "message": "[debuginfo-tests] Require \"lldb\" for some dexter tests\n\nGitOrigin-RevId: 40dd5cb11074050434db999a388a2b7aab517e4d\n" }, { "commit": "4dda378488eaa92026d3735286a243b2e859b00b", "tree": "951039bdc0feec503a846253a89def1f13880045", "parents": [ "8b479dd9404875862e6835558b70bda6cd3b7154" ], "author": { "name": "Fangrui Song", "email": "maskray@google.com", "time": "Thu Jun 25 16:49:05 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:07 2020 -0700" }, "message": "[CMake] Add llvm-config to DEBUGINFO_TEST_DEPS\n\nFix `fatal: Could not run process [\u0027/path/to/build/./bin/llvm-config\u0027, \u0027--build-mode\u0027]`\nwhen running check-debuginfo in a fresh build directory.\n\nGitOrigin-RevId: ca8b7ef763797253e08f614a7d38b76e77806b12\n" }, { "commit": "8b479dd9404875862e6835558b70bda6cd3b7154", "tree": "93a468c9dc232e48dff6650c49b1473a4cced992", "parents": [ "0538b2d1a6a4a4e37b4c488272b11b004fbcdf9d" ], "author": { "name": "Tobias Bosch", "email": "tbosch@google.com", "time": "Fri Jun 05 16:19:23 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:03 2020 -0700" }, "message": "[Dexter] Add --source-dir-root flag\n\nSummary:\nThis allows to run dexter tests with separately compiled\nbinaries that are specified via --binary if the source file\nlocation changed between compilation and dexter test run.\n\nReviewers: TWeaver, jmorse, probinson, #debug-info\n\nReviewed By: jmorse\n\nSubscribers: #debug-info, cmtice, llvm-commits\n\nTags: #llvm, #debug-info\n\nDifferential Revision: https://reviews.llvm.org/D81319\n\nGitOrigin-RevId: 53d6bfef32e371fb984a40b52208c6e34bd54e05\n" }, { "commit": "0538b2d1a6a4a4e37b4c488272b11b004fbcdf9d", "tree": "2b2310a1ca1a5891a90ed2ad79c20c703d61a2a3", "parents": [ "9edcd110390023f6a36b594c9b095d2938f673fb" ], "author": { "name": "Mehdi Amini", "email": "joker.eph@gmail.com", "time": "Fri Mar 27 23:58:06 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:01 2020 -0700" }, "message": "Change filecheck default to dump input on failure\n\nHaving the input dumped on failure seems like a better\ndefault: I debugged FileCheck tests for a while without knowing\nabout this option, which really helps to understand failures.\n\nRemove `-dump-input-on-failure` and the environment variable\nFILECHECK_DUMP_INPUT_ON_FAILURE which are now obsolete.\n\nDifferential Revision: https://reviews.llvm.org/D81422\n\nGitOrigin-RevId: d31c9e5a46ee692daf2430b52626afcea1db18ab\n" }, { "commit": "9edcd110390023f6a36b594c9b095d2938f673fb", "tree": "9de8cc763ce7bcc000e675eef00f804c6c2afd4b", "parents": [ "f960068a2a0a6a304442952c46515f1a35044e97" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Fri Jun 05 12:53:56 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:53:00 2020 -0700" }, "message": "[Dexter] Add DexLimitSteps command and ConditionalController\n\n* Adds DexLimitSteps Command.\n* Add ConditionalController, a new DebuggerController type.\n* 5 regression tests\n* documentation\n\n* recommit, fixed accidental adding of unnecessary file\n\nReviewers: jmorse\n\nDifferential Revision: https://reviews.llvm.org/D79786\n\nGitOrigin-RevId: c6aa829644f30d5590451b892918298f8117c985\n" }, { "commit": "f960068a2a0a6a304442952c46515f1a35044e97", "tree": "ca9c0cd871b5354b6beed2c8781ba9be9ef44db3", "parents": [ "086c84e863a9408db620a41c7742b8fc69b999e0" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Wed Jun 03 12:28:26 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:59 2020 -0700" }, "message": "Revert \"[Dexter] Add DexLimitSteps command and ConditionalController\"\n\nThis reverts commit 81e836a5a675f6a3d9d35560fddbbb87fdf66201.\n\nAccidentally committed a diff file.\n\nGitOrigin-RevId: 05eabb52045f3a20e137ff217b6fc1d5bd1155b4\n" }, { "commit": "086c84e863a9408db620a41c7742b8fc69b999e0", "tree": "9de8cc763ce7bcc000e675eef00f804c6c2afd4b", "parents": [ "abd15c5fb414d7554a3b0ac1fe4727f1b9ac74c4" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Tue Jun 02 16:19:43 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:58 2020 -0700" }, "message": "[Dexter] Add DexLimitSteps command and ConditionalController\n\n * Adds DexLimitSteps Command.\n * Add ConditionalController, a new DebuggerController type.\n * 5 regression tests\n * documentation\n\n Reviewers: jmorse\n\n Differential Revision: https://reviews.llvm.org/D79786\n\nGitOrigin-RevId: 81e836a5a675f6a3d9d35560fddbbb87fdf66201\n" }, { "commit": "abd15c5fb414d7554a3b0ac1fe4727f1b9ac74c4", "tree": "ca9c0cd871b5354b6beed2c8781ba9be9ef44db3", "parents": [ "8b9d72ae1f415ab7fb379930972d2181d2ca8c5d" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Tue Jun 02 16:09:17 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:54 2020 -0700" }, "message": "[Dexter] Add os.path.normcase(...) transform to test path early.\n\n When passing a test path, if the path points directly at a file, then\n normcase would not be called on path.\n\n This would change the expected lower case drive path, on windows, to be\n uppercase. This patch simply calls normcase on the test path at the earliest\n point possible to avoid this issue.\n\n Reviewers: djtodoro, jmorse\n\n Differential Revision: https://reviews.llvm.org/D78633\n\nGitOrigin-RevId: bf1cdc2c6c0460b7121ac653c796ef4995b1dfa9\n" }, { "commit": "8b9d72ae1f415ab7fb379930972d2181d2ca8c5d", "tree": "9a4db27de647d01f949c3c7ad740ab6dc17b1830", "parents": [ "b6c3f4445038d31be32d53382cd83c88f86106ce" ], "author": { "name": "Christian Sigg", "email": "csigg@google.com", "time": "Wed May 13 21:53:00 2020 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:53 2020 -0700" }, "message": "Only run pretty-printer tests for builds with debug-info.\n\nReviewers: dblaikie\n\nReviewed By: dblaikie\n\nSubscribers: tbosch, llvm-commits\n\nTags: #llvm\n\nDifferential Revision: https://reviews.llvm.org/D79897\n\nGitOrigin-RevId: 104e38cf761e3de3fb7838863001dbf007e5328b\n" }, { "commit": "b6c3f4445038d31be32d53382cd83c88f86106ce", "tree": "b0bedf2e97db53c2051fa74489ef3b1587af08b5", "parents": [ "fa85f486cc45fe1c81c18851e3a0d5001c63ad41" ], "author": { "name": "Christopher Tetreault", "email": "ctetreau@quicinc.com", "time": "Wed May 13 13:31:43 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:49 2020 -0700" }, "message": "Ensure that CMake tries to find Python3 before processing\n\nSummary:\nIt is possible that CMake tries to process debuginfo-tests before any\nattempt to find Python3. Ensure that CMake attempts to find it before\ncomplaining that it doesn\u0027t exist.\n\nReviewers: zturner, rnk, jmorse, chandlerc\n\nReviewed By: rnk\n\nSubscribers: mgorny, llvm-commits\n\nTags: #llvm\n\nDifferential Revision: https://reviews.llvm.org/D79795\n\nGitOrigin-RevId: f829ba60dadb71340318d525708e33b08ad61f8b\n" }, { "commit": "fa85f486cc45fe1c81c18851e3a0d5001c63ad41", "tree": "e3649f591ba8b9a0db96d4038300420aa7e30c7b", "parents": [ "0c90e8885137c77f57d1f6cd84838add2449a6ef" ], "author": { "name": "Jonas Devlieghere", "email": "jonas@devlieghere.com", "time": "Thu Apr 30 10:51:40 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:45 2020 -0700" }, "message": "[debuginfo-tests] Update Python variable in lit.site.cfg.py\n\nGitOrigin-RevId: 7f84b6c1ba7d3b8eabc9af406f9f7f430b057935\n" }, { "commit": "0c90e8885137c77f57d1f6cd84838add2449a6ef", "tree": "2e6b1b11f05233c076edf09afd7784ef23ff28f7", "parents": [ "d9e3227ce331a4650ea2b8b4b23afb3d01fa7afa" ], "author": { "name": "Jonas Devlieghere", "email": "jonas@devlieghere.com", "time": "Wed Apr 29 09:51:24 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:44 2020 -0700" }, "message": "[debuginfo-tests] Update Python CMake variable\n\nGitOrigin-RevId: ba8163e6290815a1deb9983772e70418d6915ac8\n" }, { "commit": "d9e3227ce331a4650ea2b8b4b23afb3d01fa7afa", "tree": "210f60758e0dc791f17e6365d631606c1625ddb9", "parents": [ "311272d372812072fa6a1651279e9d5a742f51e2" ], "author": { "name": "Djordje Todorovic", "email": "djordje.todorovic@syrmia.com", "time": "Thu Apr 23 09:49:02 2020 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:39 2020 -0700" }, "message": "[dexter] Require python \u003e\u003d 3.6\n\nThe documentation says we need python \u003e\u003d 3.6. Running it with an older\nversion, we get verbose output from python interpreter.\nThis patch fixes that as:\n\n $ python2 dexter.py list-debuggers\n You need python 3.6 or later to run DExTer\n\nDifferential Revision: https://reviews.llvm.org/D78621\n\nGitOrigin-RevId: 871388e384dc39bf7c69866ab851e2fbaff8d4c8\n" }, { "commit": "311272d372812072fa6a1651279e9d5a742f51e2", "tree": "8ac19539a208a842819f778bfad7daabeafca5bb", "parents": [ "476282041515257a5abd11f8d573db627ae435c6" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Tue Apr 21 13:40:02 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:35 2020 -0700" }, "message": "[Dexter] Fix failing clang-opt-bisect sub tool test\n\n Fixes a mismatch in expected arguments passed to run_debugger_subprocess\n\n Fix for:\n https://reviews.llvm.org/D76926\n rG9cf9710bb0d61cb5c27c6e780af6a182cb162bfb\n\nGitOrigin-RevId: b1097e29d3d06be1d5767dffddaa8f9c5dbdcc83\n" }, { "commit": "476282041515257a5abd11f8d573db627ae435c6", "tree": "c69e1f1fa8f18406d7bd94c9f7b3323da08317d4", "parents": [ "f0e76ef7f7d500f246db80ab475b7325e7d5afa5" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Mon Apr 20 15:46:55 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:34 2020 -0700" }, "message": "[Dexter][NFC] Add Debugger Controller To Dexter\n\n Add DebuggerControllerBase and DefaultController to Dexter\n implements a new architecture that supports new and novel ways of running\n a debugger under dexter.\n Current implementation adds the original default behaviour via the new\n architecture via the DefaultController, this should have NFC.\n\nReviewers: Orlando\n\nDifferential Revision: https://reviews.llvm.org/D76926\n\nGitOrigin-RevId: 9cf9710bb0d61cb5c27c6e780af6a182cb162bfb\n" }, { "commit": "f0e76ef7f7d500f246db80ab475b7325e7d5afa5", "tree": "77c604493dcf7e12223dc77f66e53630dfb31b81", "parents": [ "1bf83a8c2448298619da919dc5a24b7e4d8c4744" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Tue Mar 31 10:18:12 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:30 2020 -0700" }, "message": "[Dexter] Add support for Windows to regression test suite.\n\nThis patch addresses the issue of the regression suite not running on windows\nhardware. It changes the following things:\n\n* add new dexter regression suite command to lit.cfg.py that makes use of the\n clang-cl_vs2015 and dbgend builder and debuggers.\n\n* sprinkle the new regressionsuite command through the feature and tool tests\n that require them.\n\n* mark certain problem tests on windows\n\n* [revert fix] fixed darwin regression test failures by adding unsupported line\n for system-darwin to command tests.\n\nThere\u0027s a couple of tests that fail (or pass) in unexpected ways on Windows.\n\nProblem tests are both the penalty and perfect expect_watch_type.cpp tests.\nType information reporting parity is not possible a this time in dexter due to\nthe nature of how different debuggers report type information back to their users.\n\nreviewers: Orlando\n\nDifferential Revision: https://reviews.llvm.org/D76609\n\nGitOrigin-RevId: b6d2212f5276bcf0e5ef6b5dc0c985b72664ae34\n" }, { "commit": "1bf83a8c2448298619da919dc5a24b7e4d8c4744", "tree": "6edbb9ebcc269be1fe6bb1d02ab3c72777eb802d", "parents": [ "d7cc88bc686558f422bc85928ae15c4bb0c43a5e" ], "author": { "name": "Davide Italiano", "email": "ditaliano@apple.com", "time": "Fri Mar 27 13:12:00 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:29 2020 -0700" }, "message": "Revert \"[Dexter] Add support for Windows to regression test suite.\"\n\nThis reverts commit 89025da9f676aebff7daf055824d6fd102a70c34 as\nit breaks the lldb macOS bot.\n\nGitOrigin-RevId: 470e82cd06062a3fc3c142782b4d46d711b7c19c\n" }, { "commit": "d7cc88bc686558f422bc85928ae15c4bb0c43a5e", "tree": "66d4a85624326c20e96b83ac3707de46445c43be", "parents": [ "a25544770a1ace782b1b039b9f23911570c8668c" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Fri Mar 27 18:15:17 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:28 2020 -0700" }, "message": "[Dexter] Add support for Windows to regression test suite.\n\nThis patch addresses the issue of the regression suite not running on windows hardware. It changes the following things:\n\nadd new dexter regression suite command to lit.cfg.py that makes use of the clang-cl_vs2015 and dbgend builder and debuggers.\nsprinkle the new regression suite command through the feature and tool tests that require them.\nmark certain problem tests on windows\n\nThere\u0027s a couple of tests that fail (or pass) in unexpected ways on Windows.\n\nProblem tests are both the penalty and perfect expect_watch_type.cpp tests. Type information reporting parity is not possible a this time in dexter due to the nature of how different debuggers report type information back to their users.\n\nreviewers: Orlando\n\nDifferential Revision: https://reviews.llvm.org/D76609\n\nGitOrigin-RevId: 89025da9f676aebff7daf055824d6fd102a70c34\n" }, { "commit": "a25544770a1ace782b1b039b9f23911570c8668c", "tree": "6edbb9ebcc269be1fe6bb1d02ab3c72777eb802d", "parents": [ "25236811642097387efcf2385d39a9f4101dee8a" ], "author": { "name": "Tom Weaver", "email": "Tom.Weaver@Sony.com", "time": "Mon Mar 16 16:38:41 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:24 2020 -0700" }, "message": "[DexTer] Add step.UNKNOWN check for NoneType line numbers.\n\nSummary: It\u0027s possible for an instance of the visual studio debugger\nto return a NoneType line number location when stepping during a\ndebugging session.\n\nThis patches teaches DexTer how to handle this particular case without\ncrashing out.\n\nReviewers: Orlando\n\nDifferential revision: https://reviews.llvm.org/D75992\n\nGitOrigin-RevId: 09f4bdc03f891cbcf0c5633c8c7013eb0fbe626f\n" }, { "commit": "25236811642097387efcf2385d39a9f4101dee8a", "tree": "93caab89a2c848f6dab833f17094ebba73b6921d", "parents": [ "a68a8d0847ba5085091675f5c873f6244e3a405a" ], "author": { "name": "Jonas Devlieghere", "email": "jonas@devlieghere.com", "time": "Sun Mar 15 20:56:33 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:23 2020 -0700" }, "message": "[debuginfo-tests] Update test for double-dash long-option.\n\nGitOrigin-RevId: 54146cbd32e319029cd1ff8519f76e2a3820bb8c\n" }, { "commit": "a68a8d0847ba5085091675f5c873f6244e3a405a", "tree": "ac7d2c7d2ecf8ae9eaca840d481ca7badcae080c", "parents": [ "2928785f9db9404a0e9e44d49ecaf516f67d1f37" ], "author": { "name": "Christian Sigg", "email": "csigg@google.com", "time": "Wed Jan 29 09:57:54 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:22 2020 -0700" }, "message": "Change to individual pretty printer classes, remove generic `make_printer`.\n\nSummary: Follow-up from D72589.\n\nReviewers: dblaikie\n\nReviewed By: dblaikie\n\nSubscribers: merge_guards_bot, llvm-commits\n\nTags: #llvm\n\nDifferential Revision: https://reviews.llvm.org/D73609\n\nGitOrigin-RevId: f3ad6eb5d3d07f8e1ddb380b2b6806edd3dc84f6\n" }, { "commit": "2928785f9db9404a0e9e44d49ecaf516f67d1f37", "tree": "395b9fe28d0d7556eaf02a3dcaed75b501987e6b", "parents": [ "b4d20fc8fc3e14d9ad13a69faf7e9db89e0eb383" ], "author": { "name": "Pierre-vh", "email": "pierre.vanhoutryve@arm.com", "time": "Mon Mar 02 17:06:06 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:17 2020 -0700" }, "message": "[debuginfo-tests][dexter] Add --builder gcc support for POSIX\n\nDifferential Revision: https://reviews.llvm.org/D75339\n\nGitOrigin-RevId: b4b9fa5e11abab38e4f5dcd65b57c0226a0aee15\n" }, { "commit": "b4d20fc8fc3e14d9ad13a69faf7e9db89e0eb383", "tree": "3b3cfe1c704e0a6497e435eb963e15b316996098", "parents": [ "023de54c4e70fbc9f8d9c39ba38ac2db692bb431" ], "author": { "name": "Pierre-vh", "email": "pierre.vanhoutryve@arm.com", "time": "Wed Feb 26 14:46:51 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:13 2020 -0700" }, "message": "[debuginfo-tests][dexter] Add a test tool --calculate-average option\n\nDifferential Revision: https://reviews.llvm.org/D75235\n\nGitOrigin-RevId: 527bd24c3b41a54cef241ea2a1be53f4a4dedb17\n" }, { "commit": "023de54c4e70fbc9f8d9c39ba38ac2db692bb431", "tree": "e00a7d7315f4552bb306069198b1e2f4bef4a996", "parents": [ "f6267a256d4f8461546a71825acfcd18c7fd443c" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Tue Feb 25 13:12:13 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:09 2020 -0700" }, "message": "[debuginfo-tests] Warn, not error, if we can\u0027t delete working directory\n\nOn Windows, an error running the debugger typically leaves a process\nhanging around in the working directory. When Dexter exits, it can\u0027t then\ndelete the working directory and produces an exception, masking the problem\nin the debugger. (This can be worked around by specifying --save-temps).\nRather than hard-erroring, print a warning when we can\u0027t delete the working\ndirectory instead.\n\nIt\u0027d be much better to improve our error handling, and make the\nWorkingDirectory class aware that something\u0027s wrong when it enters exit.\nHowever, this is something that\u0027s going to mask genuine errors and make\neveryones lives harder right now, so I think this non-ideal fix is\nimportant to get in first.\n\nDifferential Revision: https://reviews.llvm.org/D74548\n\nGitOrigin-RevId: 0a3b0837915e9796ae4279fe704efa663b89d815\n" }, { "commit": "f6267a256d4f8461546a71825acfcd18c7fd443c", "tree": "62eea1126fe04719c18baef40fb0f2b3a141753a", "parents": [ "2fb27c77018e0f0d52218d9e1cb1f130cc2c32c4" ], "author": { "name": "Shoaib Meenai", "email": "smeenai@fb.com", "time": "Fri Feb 21 15:44:08 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:08 2020 -0700" }, "message": "[arcconfig] Delete subproject arcconfigs\n\nFrom https://secure.phabricator.com/book/phabricator/article/arcanist_new_project/:\n\n\u003e An .arcconfig file is a JSON file which you check into your project\u0027s root.\n\nI\u0027ve done some experimentation, and it looks like the subproject\n.arcconfigs just get ignored, as the documentation says. Given that\nwe\u0027re fully on the monorepo now, it\u0027s safe to remove them.\n\nDifferential Revision: https://reviews.llvm.org/D74996\n\nGitOrigin-RevId: e34ddc09f464667b5f3a49bf60255e89041ddf44\n" }, { "commit": "2fb27c77018e0f0d52218d9e1cb1f130cc2c32c4", "tree": "c03f75a1f4404df75ca68c1a49fc1e0c486283fa", "parents": [ "b8724284ad3796de8ba39a901b0fbeb42d423974" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Thu Feb 13 14:24:33 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:07 2020 -0700" }, "message": "[debuginfo-tests][Dexter] Fix some Windows-unfriendly Dexter behaviours\n\nThese are some minor things that I\u0027ve run into on Windows, largely in\nerror handling paths:\n\n * Giving --lldb-executable on Windows triggers a \"useless option\" code\n path, which touches an attribute that only exists in the\n list_debuggers tool. Switch this to use hasattr, which will work in\n all subtools.\n * We were over-decoding some text reporting errors, but only in an\n exception path\n * The path to lldb on Windows needs to be quoted (even though dexter\n isn\u0027t making use of it).\n\nDifferential Revision: https://reviews.llvm.org/D74546\n\nGitOrigin-RevId: 5ee4a03bc916d2925b9431c53b7da0d3ce7e2b9e\n" }, { "commit": "b8724284ad3796de8ba39a901b0fbeb42d423974", "tree": "f87a8696e781d6f52210cef67dbc10c74a08eb14", "parents": [ "25e670462765bbb52ab7a22358a1f97262c74672" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Thu Feb 13 12:43:28 2020 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:52:03 2020 -0700" }, "message": "[debuginfo-tests] Fix Dexter process creation failure on Windows\n\nWhen writing the Windows dbgeng driver for Dexter, I couldn\u0027t work out why it\nwould either launch a process and leave it free running, or if I started the\nprocess suspended, never do anything with it. The result was a hack to create\nand attach processes manually. This has been flaking out on Reids Windows\nbuildbot, and clearly wasn\u0027t a good solution.\n\nDigging into this, it turns out that the \"normal\" cdb / windbg behaviour of\nbreaking whenever we attach to a process is not the default: it has to be\nexplicitly requested from the debug engine. This patch does so (by setting\nDEBUG_ENGOPT_INITIAL_BREAK in the engine options), after which we can simply\ncall \"CreateProcessAndAttach2\" and everything automagically works.\n\nNo test for this behaviour: everything was just broken before.\n\nDifferential Revision: https://reviews.llvm.org/D74409\n\nGitOrigin-RevId: 26f6aa9e3ba47c5ccb9d837310b59a5087549b7d\n" }, { "commit": "25e670462765bbb52ab7a22358a1f97262c74672", "tree": "588d80e154d35c916dd20e777412b192f0033d34", "parents": [ "6ef1aeb3b345e4244550efc2f1ce1957e9bb8f7d" ], "author": { "name": "Adrian Prantl", "email": "aprantl@apple.com", "time": "Fri Feb 07 15:09:44 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:58 2020 -0700" }, "message": "Replace CHECK-NEXT with CHECK-DAG. The order isn\u0027t relevant we just\nwant to make sure that all are present.\n\nGitOrigin-RevId: 248435e9c9e62fd95f5b38ae9ba0dfdb051929cb\n" }, { "commit": "6ef1aeb3b345e4244550efc2f1ce1957e9bb8f7d", "tree": "861d2cddedf9f41b990796610d664d451a317b89", "parents": [ "83ce51a90c82778a4c45c21c613d1ba11b70707f" ], "author": { "name": "Jason Molenda", "email": "jmolenda@apple.com", "time": "Fri Feb 07 14:31:35 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:54 2020 -0700" }, "message": "Change first test to be CHECK: to make the test run.\n\nGitOrigin-RevId: 4a18a5bdd9e2e119cf24653e34b22cf356fffe77\n" }, { "commit": "83ce51a90c82778a4c45c21c613d1ba11b70707f", "tree": "f4b0e515ae9ecde6d3228de1dd01f9c848833d70", "parents": [ "a3fbb7edfd556f864854ebecfdbe3dbe3a8c5694" ], "author": { "name": "Sourabh Singh Tomar", "email": "SourabhSingh.Tomar@amd.com", "time": "Sat Feb 08 01:24:04 2020 +0530" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:49 2020 -0700" }, "message": "[DebugInfo]: Fix the debuginfo-tests/llgdb-tests/apple-accel.cpp test\nfailure after 84e5760.\n\nGitOrigin-RevId: 5752bcf9e5ace8dc4942ba2e044ec045b8da7eb0\n" }, { "commit": "a3fbb7edfd556f864854ebecfdbe3dbe3a8c5694", "tree": "015aca44b58f12ef56359c0c093ebbcacdc7cd97", "parents": [ "61baf2f7f7e56f96a9592f98e14487e10467048c" ], "author": { "name": "Christian Sigg", "email": "csigg@google.com", "time": "Fri Jan 31 12:19:36 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:45 2020 -0700" }, "message": "Rename prettyprinters test to llvm-support.\n\nSummary: Make room for mlir-support pretty printers that I would like to add next.\n\nReviewers: dblaikie\n\nReviewed By: dblaikie\n\nSubscribers: merge_guards_bot, mgorny, rriddle, llvm-commits\n\nTags: #llvm\n\nDifferential Revision: https://reviews.llvm.org/D73726\n\nGitOrigin-RevId: 5a21d2bbb982e431fe0512af2438a1a5e1e13c29\n" }, { "commit": "61baf2f7f7e56f96a9592f98e14487e10467048c", "tree": "7778771d0c61c6cdbc4b95466065ede95567af28", "parents": [ "7389c1cb654508b1df3495fa72a445b48993401c" ], "author": { "name": "Christian Sigg", "email": "csigg@google.com", "time": "Tue Jan 28 22:02:17 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:41 2020 -0700" }, "message": "Add GDB pretty printers for llvm::ilist, llvm::simple_ilist, and llvm::ilist_node.\n\nReviewers: dblaikie\n\nReviewed By: dblaikie\n\nSubscribers: merge_guards_bot, llvm-commits\n\nTags: #llvm\n\nDifferential Revision: https://reviews.llvm.org/D72589\n\nGitOrigin-RevId: a03ec58da60c044dc3be7cbb0fcd7912769b05ef\n" }, { "commit": "7389c1cb654508b1df3495fa72a445b48993401c", "tree": "b296eb99ccc07fabe4e7af5a2b53e027b682a4d4", "parents": [ "8d2575d83a73b0e3999e6e58e717245f1ceda21d" ], "author": { "name": "Christian Sigg", "email": "csigg@google.com", "time": "Mon Jan 27 17:23:39 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:37 2020 -0700" }, "message": "Add pretty printers for llvm::PointerIntPair and llvm::PointerUnion.\n\nReviewers: aprantl, dblaikie, jdoerfert, nicolasvasilache\n\nReviewed By: dblaikie\n\nSubscribers: jpienaar, dexonsmith, merge_guards_bot, llvm-commits\n\nTags: #llvm, #clang, #lldb, #openmp\n\nDifferential Revision: https://reviews.llvm.org/D72557\n\nGitOrigin-RevId: 97431831e5690275a453567430d1153c47ba1585\n" }, { "commit": "8d2575d83a73b0e3999e6e58e717245f1ceda21d", "tree": "c4f72b4bb5616e6d5bd055793ef377a15e26e5a5", "parents": [ "dcd7ae4f79235517d179efa550253332a1d28eb7" ], "author": { "name": "Christian Sigg", "email": "csigg@google.com", "time": "Sat Jan 11 08:47:41 2020 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:33 2020 -0700" }, "message": "Add test for GDB pretty printers.\n\nReviewers: dblaikie, aprantl, davide, JDevlieghere\n\nReviewed By: aprantl\n\nSubscribers: jmorse, aprantl, merge_guards_bot, mgorny, llvm-commits\n\nTags: #llvm\n\nDifferential Revision: https://reviews.llvm.org/D72321\n\nGitOrigin-RevId: 60346bdbd73da9c944d50ea5dcecad71a05105ac\n" }, { "commit": "dcd7ae4f79235517d179efa550253332a1d28eb7", "tree": "5d5687ed0f1fb32f9465f955ca0c233403ef7799", "parents": [ "9c543579e5187100260f5a9925955b81845a8137" ], "author": { "name": "Jonas Devlieghere", "email": "jonas@devlieghere.com", "time": "Mon Nov 11 10:40:06 2019 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:28 2020 -0700" }, "message": "[debuginfo] Update test to account for missing __debug_macinfo\n\nWe no longer emit this section if it\u0027s empty.\n\nGitOrigin-RevId: 8a44d4850d268fdcc61b95fdd32233b7a3a15e9b\n" }, { "commit": "9c543579e5187100260f5a9925955b81845a8137", "tree": "f637fb57d8426729261fab8fe2940f0cb435342f", "parents": [ "2341974cde7da51ce5fc3954e39938a31c3f823c" ], "author": { "name": "Adrian Prantl", "email": "aprantl@apple.com", "time": "Fri Nov 08 14:09:40 2019 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:24 2020 -0700" }, "message": "Don\u0027t attempt to upgrade debug-info-tests to Python3.\n\nThere was CMake code that would attempt to detect Python 3 even if\nLLVM was configured with Python 2, but it messed with the environment\nfor the other LLVM projects. This commit removes this functionality\nand just lkeaves a hard error if Python \u003c 3 is detected and\ndebuginfo-tests are required.\n\nGitOrigin-RevId: d0416b91f0390f6e69dacf3d5d076531221e0767\n" }, { "commit": "2341974cde7da51ce5fc3954e39938a31c3f823c", "tree": "7683c020723835ceb4bb68627bc64cb4d985764e", "parents": [ "952789b97f7ede323555419da88ba0e0f11a3624" ], "author": { "name": "Reid Kleckner", "email": "rnk@google.com", "time": "Fri Nov 01 11:47:53 2019 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:20 2020 -0700" }, "message": "[dexter] Fix feature tests on Windows\n\nFirst, add LLD as a dependency on Windows. The windows batch scripts\npass -fuse-ld\u003dlld, so they need it.\n\nSecond, decode builder stdout/stderr even if the command fails.\nOtherwise it gets printed as b\u0027line 1\\n\\rline 2\\n\\r\u0027.\n\nLast, make the batch script one line less noisy. We might want to try to\ndo more here, though. It would be nice if we could get as close to\npossible as lit, where you can literally copy \u0026 paste the failing\ncommand to re-run it.\n\nWith the two changes above, now the feature tests that use clang++.bat\npass for me. The clang-cl_vs2015 ones still fail, and I\u0027ll fix them\nseparately.\n\nReviewers: jmorse\n\nDifferential Revision: https://reviews.llvm.org/D69725\n\nGitOrigin-RevId: 63f49465c3268e59ce2dc02345da187393adabd7\n" }, { "commit": "952789b97f7ede323555419da88ba0e0f11a3624", "tree": "04f5b11a6a393a7d1ef84f20bc2078f76aa91895", "parents": [ "e3767a22747db7a47f6e49b80b3def445b029777" ], "author": { "name": "Reid Kleckner", "email": "rnk@google.com", "time": "Fri Nov 01 11:34:02 2019 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:15 2020 -0700" }, "message": "[dexter] Remove lit check for python 3\n\nThis is checking the version of Python used to run lit, which is not\nnecessarily the same as the version used to run the dexter tests. If\nthe tests are run via the build/bin/llvm-lit[.py] helper script, then\nthat is likely to pick up whatever version of Python is on PATH.\nConventionally, this will find Python 2. CMake already checks that\nPython 3 is in use and puts the path to it in the lit site config, so\nthis check is redundant, and Python 3 will ultimately be used to run\ndexter.\n\nReviewers: jmorse\n\nDifferential Revision: https://reviews.llvm.org/D69724\n\nGitOrigin-RevId: 7035ea6e3e4371fff223a3a8f660ec149a954876\n" }, { "commit": "e3767a22747db7a47f6e49b80b3def445b029777", "tree": "212d5cb657a720b538794b51c7079d945bd646c5", "parents": [ "0c2476bce2bec56ee42d17839add23c3997e6adc" ], "author": { "name": "Reid Kleckner", "email": "rnk@google.com", "time": "Thu Oct 31 14:00:22 2019 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:14 2020 -0700" }, "message": "[debuginfo-tests] Don\u0027t look for Python 3 if we already have it\n\nLLDB already requires Python 3 on Windows, so I already configure it\nthat way. For some reason CMake fails to find the one that Visual Studio\nautomatically installs at this standard location:\n C:/Program Files (x86)/Microsoft Visual Studio/Shared/Python37_64/python.exe\n\nCMake prefers the python on path, which happens to be python 2.7.\n\nReviewers: aprantl, jmorse\n\nDifferential Revision: https://reviews.llvm.org/D69684\n\nGitOrigin-RevId: 309f9a1c52977cbd011b29909f7aa45c124b005f\n" }, { "commit": "0c2476bce2bec56ee42d17839add23c3997e6adc", "tree": "fcd83c0d6ba2c5cf3a943025011049a191b4c7c0", "parents": [ "568f065d69d323e77a2402e5cbc2a22823a36c66" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Fri Nov 01 13:56:31 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:10 2020 -0700" }, "message": "[Dexter] Continue sprinking no-location fixes\n\nExample failure:\n\nhttp://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/3255/testReport/junit/debuginfo-tests/dexter-tests/asan_c/\n\nThe tests themselves seem to be working, it\u0027s just unexplored paths within\ndexter that are flaking out.\n\nGitOrigin-RevId: be79db93fbd13877ccefb9a7edbabd0d490d86c4\n" }, { "commit": "568f065d69d323e77a2402e5cbc2a22823a36c66", "tree": "a61c02f8552b627dd3d4b48b891e09acdbdc72d1", "parents": [ "311d92db07d0894de5e1ee958b647956b34e593e" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Fri Nov 01 13:41:56 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:09 2020 -0700" }, "message": "[Dexter] Account for another no-lineno scenario\n\nThis is another part of Dexter that had never seen a missing source\nlocation before, now newly turning up on Darwin.\n\nGitOrigin-RevId: 6e217ba3ea9f5730d477facf8164eeb789016a4b\n" }, { "commit": "311d92db07d0894de5e1ee958b647956b34e593e", "tree": "8684e10a81597024c9f5876e1cedac47c7e493e1", "parents": [ "c3941c56f6930728080097ebd63bfbe0d025ff82" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Fri Nov 01 13:20:26 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:05 2020 -0700" }, "message": "[Dexter] Cope better with empty source locations\n\nWhen running a program, Dexter single steps if it\u0027s in one of the source\nfiles under test, or free-runs if it isn\u0027t. Handle the circumstance where\nthe current source file simply isn\u0027t known.\n\nGitOrigin-RevId: e34179aad165fdab060d2da52703f9353d490911\n" }, { "commit": "c3941c56f6930728080097ebd63bfbe0d025ff82", "tree": "b3590435cdd92b6288ca6c3c2e400aa01440a094", "parents": [ "68fbba891d2fe3402336558e969ca4e2cc99a25b" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Fri Nov 01 13:09:37 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:51:00 2020 -0700" }, "message": "Unmask dexter debuginfo tests on Darwin\n\nThese tests almost certainly work on Darwin anyway, I just wanted to\nkeep things in a fixed, working configuration, while pushing Dexter\nup.\n\nI\u0027ve left Windows unsupported as the dexter command line will need further\nadjustment to run dbgeng. This can be abstracted through the %dexter\nsubstitution, but is a task for another time.\n\nGitOrigin-RevId: 2665f3e4364f4c3aca155e16070315f059a21b41\n" }, { "commit": "68fbba891d2fe3402336558e969ca4e2cc99a25b", "tree": "876d02c152108c42cb0dd46a7ed964af2c087532", "parents": [ "6fe30b820064cfccb86fc6e925a9afcfb2b54ddd" ], "author": { "name": "Jeremy Morse", "email": "jeremy.morse@sony.com", "time": "Fri Nov 01 12:29:42 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:50:59 2020 -0700" }, "message": "Fix a brain-fail with debuginfo-tests/dexter internal tests\n\nI baked the \"test\" subcommand into the %dexter substituion, as that\u0027s\nwhat all of the dexter tests use. However I forgot about the internal\ntests for whether dexters features are working. Install a %dexter_base\ncommand to allow those tests to call dexter.py directly, and un-xfail\nthe tests on darwin.\n\nUpdate too the list of paths the unittests shouldn\u0027t try and cover, as it\ntries to load dbgeng on unix machines. Ideally we wouldn\u0027t be using this\nmethod of test discovery in the future.\n\nGitOrigin-RevId: 7f738c811ac434fc70658cd5ed39b9a36605b561\n" }, { "commit": "6fe30b820064cfccb86fc6e925a9afcfb2b54ddd", "tree": "14fb350221fd7f62b233e535e267063ed9a90407", "parents": [ "87bb7e456b55750c5b86414c0b149144cb14d4bb" ], "author": { "name": "Reid Kleckner", "email": "rnk@google.com", "time": "Thu Oct 31 15:23:45 2019 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:50:55 2020 -0700" }, "message": "Quote path to Python 3 executable in case it contains spaces\n\nGitOrigin-RevId: f9061049c1542579f661c3ec25528439ab9d0588\n" }, { "commit": "87bb7e456b55750c5b86414c0b149144cb14d4bb", "tree": "cb28ccbcb943c14e45489c15836b828c8ee9235e", "parents": [ "aa417508254ba826ebb6bca4032e9d6355ba7db5" ], "author": { "name": "Reid Kleckner", "email": "rnk@google.com", "time": "Thu Oct 31 15:18:54 2019 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:50:51 2020 -0700" }, "message": "Fix undefined variable bug in debuginfotests lit.cfg.py\n\nGitOrigin-RevId: 7ac4c31762fbe1605e2e701e4b48698f0f95a5c7\n" }, { "commit": "aa417508254ba826ebb6bca4032e9d6355ba7db5", "tree": "bead4ae1de01553237fe267f688bd8fd6af639d3", "parents": [ "b08fc9aa30b2592a00527e98fe3a0c46e5940d44" ], "author": { "name": "Adrian Prantl", "email": "aprantl@apple.com", "time": "Thu Oct 31 11:22:19 2019 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun Aug 30 13:50:47 2020 -0700" }, "message": "XFAIL dexter tests on Darwin.\n\nThese fail parsing command line arguments.\n\nGitOrigin-RevId: 8e406204418895f7b09d1a9a3f8037e741a43968\n" } ], "next": "b08fc9aa30b2592a00527e98fe3a0c46e5940d44" }