[Lldb-commits] [PATCH] D110535: [llvm] [ADT] Update llvm::Split() per Pavel Labath's suggestions

Michał Górny via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 27 06:18:00 PDT 2021


mgorny updated this revision to Diff 375238.
mgorny marked an inline comment as done.
mgorny added a comment.
Herald added a subscriber: hiraditya.

Make the function lowercase. I had to modify `IR/DataLayout.cpp` to call its own static `split()` via `::split()`.


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

https://reviews.llvm.org/D110535

Files:
  lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
  lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp
  lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
  llvm/include/llvm/ADT/StringExtras.h
  llvm/lib/IR/DataLayout.cpp
  llvm/unittests/ADT/StringExtrasTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D110535.375238.patch
Type: text/x-patch
Size: 8938 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20210927/b0ba2d2a/attachment.bin>


More information about the lldb-commits mailing list