Files
tubestation/mozglue/baseprofiler/core/VTuneProfiler.cpp
Markus Stange 7aa7a7247f Bug 1634784 - Remove #ifdef MOZ_GECKO_PROFILER wrappers in mozglue/baseprofiler cpp files, and reorder includes. r=gerald
I apologize for mixing multiple types of changes in this patch. The mix is probably harder to review than necessary.

Because of the way these .cpp files are listed in mozglue/baseprofiler/moz.build,
they are only built when MOZ_GECKO_PROFILER is set. So the #ifdef MOZ_GECKO_PROFILER
wrapper is now unnecessary.
Removing this wrapper has many knock-on effects on other lines in these files
due to preprocessor line indentation.

As I was removing the #ifdefs, I was reordering includes as well, to follow the
include order laid out on https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes ;
namely, system headers should go before "your project's" .h files.

I also removed the BaseProfiler.h include from some files but not from all.
In the past, the BaseProfiler.h include served the purpose of picking up the
MOZ_BASE_PROFILER #define, and it's no longer needed for that reason.
There are probably lots of now-unnecessary BaseProfiler.h includes left.

Differential Revision: https://phabricator.services.mozilla.com/D73527
2020-05-05 22:00:49 +00:00

93 lines
2.7 KiB
C++

/* -*- Mode: C++; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
/* vim: set ts=8 sts=2 et sw=2 tw=80: */
/* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
#ifdef XP_WIN
# undef UNICODE
# undef _UNICODE
#endif
#include "VTuneProfiler.h"
#include <memory>
#include "BaseProfiler.h"
namespace mozilla {
namespace baseprofiler {
VTuneProfiler* VTuneProfiler::mInstance = nullptr;
void VTuneProfiler::Initialize() {
// This is just a 'dirty trick' to find out if the ittnotify DLL was found.
// If it wasn't this function always returns 0, otherwise it returns
// incrementing numbers, if the library was found this wastes 2 events but
// that should be okay.
// TODO re-implement here if vtune is needed
// __itt_event testEvent =
// __itt_event_create("Test event", strlen("Test event"));
// testEvent = __itt_event_create("Test event 2", strlen("Test event 2"));
// if (testEvent) {
// mInstance = new VTuneProfiler();
// }
}
void VTuneProfiler::Shutdown() {}
void VTuneProfiler::TraceInternal(const char* aName, TracingKind aKind) {
// TODO re-implement here if vtune is needed
// std::string str(aName);
// auto iter = mStrings.find(str);
// __itt_event event;
// if (iter != mStrings.end()) {
// event = iter->second;
// } else {
// event = __itt_event_create(aName, str.length());
// mStrings.insert({str, event});
// }
// if (aKind == TRACING_INTERVAL_START || aKind == TRACING_EVENT) {
// // VTune will consider starts not matched with an end to be single point
// in
// // time events.
// __itt_event_start(event);
// } else {
// __itt_event_end(event);
// }
}
void VTuneProfiler::RegisterThreadInternal(const char* aName) {
// TODO re-implement here if vtune is needed
// std::string str(aName);
// if (!str.compare("Main Thread (Base Profiler)")) {
// // Process main thread.
// switch (XRE_GetProcessType()) {
// case GeckoProcessType::GeckoProcessType_Default:
// __itt_thread_set_name("Main Process");
// break;
// case GeckoProcessType::GeckoProcessType_Content:
// __itt_thread_set_name("Content Process");
// break;
// case GeckoProcessType::GeckoProcessType_GMPlugin:
// __itt_thread_set_name("Plugin Process");
// break;
// case GeckoProcessType::GeckoProcessType_GPU:
// __itt_thread_set_name("GPU Process");
// break;
// default:
// __itt_thread_set_name("Unknown Process");
// }
// return;
// }
// __itt_thread_set_name(aName);
}
} // namespace baseprofiler
} // namespace mozilla