[llvm] [llvm-exegesis][NFC] Refactor all `ValidationEvent` info in a single … (PR #82256)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 20 13:47:16 PST 2024


================
@@ -0,0 +1,60 @@
+//===-- ValidationEvent.h ---------------------------------------*- C++ -*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+///
+/// \file
+/// Definitions and utilities for Validation Events.
+///
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_TOOLS_LLVM_EXEGESIS_VALIDATIONEVENT_H
+#define LLVM_TOOLS_LLVM_EXEGESIS_VALIDATIONEVENT_H
+
+#include "llvm/ADT/StringRef.h"
+#include "llvm/Support/Error.h"
+
+namespace llvm {
+
+namespace exegesis {
+
+// The main list of supported validation events. The mapping between validation
+// events and pfm counters is defined in TableDef files for each target.
----------------
boomanaiden154 wrote:

Nit: Maybe add a note here that the text mapping in `ValidationEvent.cpp` needs to be updated as well? The `static_assert` should take care of it, but having the info in the comment wouldn't hurt.

https://github.com/llvm/llvm-project/pull/82256


More information about the llvm-commits mailing list