[llvm-branch-commits] [llvm] [CGData] Refactor Global Merge Functions (PR #115750)

[Ellis Hoag via llvm-branch-commits]

ellishg wrote:

> Hit an assertion in `ignoreOp` when testing the refactored code.
> 
> ```
> Assertion failed: (OpIdx < I->getNumOperands() && "Invalid operand index"), function ignoreOp, file GlobalMergeFunctions.cpp, line 129.
> Stop reason: hit program assert
> expr I->dump()
>   %6 = tail call ptr @objc_retain(ptr %5), !dbg !576
> 
> p I->getNumOperands()
> 
> (unsigned int) 2
> p OpIdx
> 
> (unsigned int) 4
> ```

Do we know why `OpIdx` is 4 here? This is confusing to me because it looks like there is only one argument, `%5`.

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