[PATCH] D31987: [SDAG] Teach Chain Analysis about BaseIndexOffset addressing.

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

I had not looked at this code before, so I'll need help reviewing. As part of trying to understand what this function is doing, I tried to improve it with some cleanups:
https://reviews.llvm.org/rL300850
https://reviews.llvm.org/rL300854
https://reviews.llvm.org/rL300860

This patch says that if BaseIndexOffset::match() returns matching base pointers, then we don't need to go through all of the checks below it. In that case, can we remove some/all of the code in findBaseOffset()?


https://reviews.llvm.org/D31987