<div dir="ltr">Hi Sean,<div><br></div><div>A change went in over the weekend to change the way GetElementPtrInst::Create works.  Now, for pointer types, you need to manually specify the type of the pointee, as pointerType->getType() won't work anymore.  This broke the build, so a workaround was employed to simply pass nullptr for this argument.  But this is likely to be incorrect, and instead we need the actual llvm::Type for the pointee.  I've looked over the code, and I *think* that on line 2209 we can replace the nullptr with old_constant->getType(), and on line 2396 we can place the nullptr with value->getType().</div><div><br></div><div>What are your thoughts here?</div><div><br></div><div>Secondly, the change to nullptr does not seem to have caused any test failures.  Is there any way we can add some tests for this?  If this is not something that is testable through the public interface, I recently added gtest unit tests to the build.  It's not enabled in the Xcode build (but will be soon), but if you tell me how to set up a test, call the right function, and what outputs to expect, I can write it.</div></div>