[lldb] Fix 7b73de9ec2b19df040c919d3004dfbead9b6ac59
authorWalter Erquinigo <wallace@fb.com>
Tue, 10 May 2022 04:12:09 +0000 (21:12 -0700)
committerWalter Erquinigo <wallace@fb.com>
Tue, 10 May 2022 04:12:11 +0000 (21:12 -0700)
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.

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

index 30c30eb..a1cd30b 100644 (file)
@@ -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);