[Lldb-commits] [lldb] 45e6e4d - [lldb][NFCI] Remove unneeded ConstString from intel-pt plugin (#66721)

via lldb-commits lldb-commits at lists.llvm.org
Tue Sep 19 10:49:42 PDT 2023


Author: Alex Langford
Date: 2023-09-19T10:49:38-07:00
New Revision: 45e6e4d70e1e5bdd7d9f04dfd0f4cc8798bcde0c

URL: https://github.com/llvm/llvm-project/commit/45e6e4d70e1e5bdd7d9f04dfd0f4cc8798bcde0c
DIFF: https://github.com/llvm/llvm-project/commit/45e6e4d70e1e5bdd7d9f04dfd0f4cc8798bcde0c.diff

LOG: [lldb][NFCI] Remove unneeded ConstString from intel-pt plugin (#66721)

This is a follow-up to 14d95b26aee0ac0ac8a70252e8a3c7a986e0e812. I would
have changed it in that commit, but I don't build the intel-pt plugin so
I didn't see this until later.

Added: 
    

Modified: 
    lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.cpp
    lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.h

Removed: 
    


################################################################################
diff  --git a/lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.cpp b/lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.cpp
index bcac731713bb24b..015faa89fcfada8 100644
--- a/lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.cpp
+++ b/lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.cpp
@@ -48,8 +48,8 @@ enum {
 #include "TraceIntelPTPropertiesEnum.inc"
 };
 
-ConstString TraceIntelPT::PluginProperties::GetSettingName() {
-  return ConstString(TraceIntelPT::GetPluginNameStatic());
+llvm::StringRef TraceIntelPT::PluginProperties::GetSettingName() {
+  return TraceIntelPT::GetPluginNameStatic();
 }
 
 TraceIntelPT::PluginProperties::PluginProperties() : Properties() {

diff  --git a/lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.h b/lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.h
index 20faabdce790241..da9cefe9ed95d67 100644
--- a/lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.h
+++ b/lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.h
@@ -26,7 +26,7 @@ class TraceIntelPT : public Trace {
   /// Properties to be used with the `settings` command.
   class PluginProperties : public Properties {
   public:
-    static ConstString GetSettingName();
+    static llvm::StringRef GetSettingName();
 
     PluginProperties();
 


        


More information about the lldb-commits mailing list