[PATCH] D38355: [X86][NFC] Add X86CmovConverterPass to the pass registry

[Amjad Aboud via Phabricator via llvm-commits]

aaboud marked an inline comment as done.
aaboud added inline comments.


================
Comment at: lib/Target/X86/X86CmovConversion.cpp:84
+  X86CmovConverterPass() : MachineFunctionPass(ID) {
+    initializeX86CmovConverterPassPass(*PassRegistry::getPassRegistry());
+  }
----------------
aaboud wrote:
> craig.topper wrote:
> > craig.topper wrote:
> > > Maybe do this in LLVMInitializeX86Target in X86TargetMachine.cpp? That's where we're doing the other passes that are registered.
> > Or maybe we need to do it in both places?
> Cannot do in both places, you can register a pass only once.
> I moved the registration to the LLVMInitializeX86Target as you suggested.
Sorry, my mistake, we can call it twice.
Uploaded a new patch.


https://reviews.llvm.org/D38355