[all-commits] [llvm/llvm-project] 0f362f: [lldb] Remove process->LoadModules call from Dynam...

[Pavel Labath via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0f362f9d25fd7b4baca56d3c76c688ca0113813a
      https://github.com/llvm/llvm-project/commit/0f362f9d25fd7b4baca56d3c76c688ca0113813a
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2022-03-18 (Fri, 18 Mar 2022)

  Changed paths:
    M lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp

  Log Message:
  -----------
  [lldb] Remove process->LoadModules call from DynamicLoaderPOSIXDYLD

The call is useless, as any modules loaded there will be removed in
ResolveExecutableModule. Modules will be reloaded again through the
GetLoadedModuleList call in DYLDRendezvous.cpp.


  Commit: c7fc7205bb45ee24193c519e9cd2f9f64bdde4ed
      https://github.com/llvm/llvm-project/commit/c7fc7205bb45ee24193c519e9cd2f9f64bdde4ed
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2022-03-18 (Fri, 18 Mar 2022)

  Changed paths:
    M lldb/docs/resources/test.rst

  Log Message:
  -----------
  [lldb] Add more documentation on test variants

This formalizes some of the discussion on D121631.

Differential Revision: https://reviews.llvm.org/D121900


  Commit: ab25757522341f966bf088912dcc6f9303de3067
      https://github.com/llvm/llvm-project/commit/ab25757522341f966bf088912dcc6f9303de3067
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2022-03-18 (Fri, 18 Mar 2022)

  Changed paths:
    M llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h

  Log Message:
  -----------
  Remove a top-level "using namespace" directive from LegalizationArtifactCombiner.h

The directive pollutes the namespace of all files including that header.
Move the directive into the bodies of functions that need it instead.


  Commit: 53e9ee3027dbbee79585335f608bcf5b8f668d0a
      https://github.com/llvm/llvm-project/commit/53e9ee3027dbbee79585335f608bcf5b8f668d0a
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2022-03-18 (Fri, 18 Mar 2022)

  Changed paths:
    R lldb/lldb/test/API/iohandler/sigint/Makefile
    R lldb/lldb/test/API/iohandler/sigint/TestProcessIOHandlerInterrupt.py
    R lldb/lldb/test/API/iohandler/sigint/cat.cpp

  Log Message:
  -----------
  Revert "[lldb] Fix ^C handling in IOHandlerProcessSTDIO"

This reverts commit f93d861349f923f6b7ca1a425d3632eec1ff2a72 -- looks
like a bad application of `patch -p<level>`


  Commit: 14bd14f9f92f9f5eff43ec3a977b127dea1cb521
      https://github.com/llvm/llvm-project/commit/14bd14f9f92f9f5eff43ec3a977b127dea1cb521
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2022-03-18 (Fri, 18 Mar 2022)

  Changed paths:
    M lldb/source/Target/Process.cpp
    A lldb/test/API/iohandler/sigint/Makefile
    A lldb/test/API/iohandler/sigint/TestProcessIOHandlerInterrupt.py
    A lldb/test/API/iohandler/sigint/cat.cpp

  Log Message:
  -----------
  [lldb] Fix ^C handling in IOHandlerProcessSTDIO

D120762 accidentally moved the interrupt check into the block which was
reading stdio. This meant that a ^C only took effect after a regular
character has been pressed.

This patch fixes that and adds a (pexpect) test.

Differential Revision: https://reviews.llvm.org/D121912


Compare: https://github.com/llvm/llvm-project/compare/d6d9a8b8a141...14bd14f9f92f