[llvm] [llvm-exegesis] Set stack pointer register after starting perf counter (PR #72489)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 28 23:51:32 PST 2023


================
@@ -170,6 +170,12 @@ class ExegesisTarget {
         "configurePerfCounter is not implemented on the current architecture");
   }
 
+  // Returns the stack register for the platform.
+  virtual unsigned getStackRegister() const {
+    report_fatal_error(
+        "getStackRegister is not implemented on the current architecture");
+  }
----------------
boomanaiden154 wrote:

That is definitely a lot cleaner. Thanks for the suggestion! I went through `BasicBlockFillers` `MachineFunction` rather than `LLVMState` as it was already available and seems about as clean as `generateSnippetSetupCode` doesn't already have a `LLVMState` parameter. Can change if desired though.

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


More information about the llvm-commits mailing list