[Lldb-commits] [lldb] c4172c7 - [lldb] Fix 7b73de9ec2b19df040c919d3004dfbead9b6ac59

Walter Erquinigo via lldb-commits lldb-commits at lists.llvm.org
Mon May 9 21:13:34 PDT 2022


Author: Walter Erquinigo
Date: 2022-05-09T21:12:11-07:00
New Revision: c4172c751a39313efd9c7588a103482c3648987e

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

LOG: [lldb] Fix 7b73de9ec2b19df040c919d3004dfbead9b6ac59

It turns out that the issue in
https://lab.llvm.org/buildbot/#/builders/17/builds/21754 is that a
size_t is attempted to be used interchangeably with uint64_t.

Added: 
    

Modified: 
    lldb/source/Plugins/Process/Linux/IntelPTSingleBufferTrace.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/source/Plugins/Process/Linux/IntelPTSingleBufferTrace.cpp b/lldb/source/Plugins/Process/Linux/IntelPTSingleBufferTrace.cpp
index 30c30eb6cfff..a1cd30b2fb5a 100644
--- a/lldb/source/Plugins/Process/Linux/IntelPTSingleBufferTrace.cpp
+++ b/lldb/source/Plugins/Process/Linux/IntelPTSingleBufferTrace.cpp
@@ -179,9 +179,7 @@ GeneratePerfEventConfigValue(bool enable_tsc, Optional<size_t> psb_period) {
 ///   or an \a llvm::Error otherwise.
 static Expected<perf_event_attr>
 CreateIntelPTPerfEventConfiguration(bool enable_tsc,
-                                    llvm::Optional<size_t> psb_period) {
-  return createStringError(inconvertibleErrorCode(),
-                           "Intel PT Linux perf event not supported");
+                                    llvm::Optional<uint64_t> psb_period) {
   perf_event_attr attr;
   memset(&attr, 0, sizeof(attr));
   attr.size = sizeof(attr);


        


More information about the lldb-commits mailing list