[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