[llvm] [Instrumentor] Use OptionalPassInfoMixin (PR #196333)

via llvm-commits llvm-commits at lists.llvm.org
Thu May 7 07:48:34 PDT 2026


llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-transforms

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

This preserves the existing behavior and makes the pass compliant with \#<!-- -->192120, which is important as inheriting from the normal PassInfoMixin will be going away soonish.

---
Full diff: https://github.com/llvm/llvm-project/pull/196333.diff


1 Files Affected:

- (modified) llvm/include/llvm/Transforms/IPO/Instrumentor.h (+1-1) 


``````````diff
diff --git a/llvm/include/llvm/Transforms/IPO/Instrumentor.h b/llvm/include/llvm/Transforms/IPO/Instrumentor.h
index c115bea24bb27..519a01a2626bd 100644
--- a/llvm/include/llvm/Transforms/IPO/Instrumentor.h
+++ b/llvm/include/llvm/Transforms/IPO/Instrumentor.h
@@ -678,7 +678,7 @@ struct LoadIO : public InstructionIO<Instruction::Load> {
 } // namespace instrumentor
 
 /// The Instrumentor pass.
-class InstrumentorPass : public PassInfoMixin<InstrumentorPass> {
+class InstrumentorPass : public OptionalPassInfoMixin<InstrumentorPass> {
   using InstrumentationConfig = instrumentor::InstrumentationConfig;
   using InstrumentorIRBuilderTy = instrumentor::InstrumentorIRBuilderTy;
 

``````````

</details>


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


More information about the llvm-commits mailing list