[PATCH] D16530: [Polly] Introduce MemAccInst helper class; NFC

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 09:13:47 PST 2016


grosser added a comment.

Hi Michael,

this looks a lot better. Johannes pointed out to me that there is already a function polly::getPointerOperand in ScopHelper.cpp. Having both is probably redundant. I think the GetElementPtrInst is never used in this function (please check). So we can probably eliminate it and just use your class instead of the getPointerOperand instruction. Otherwise, this is probably good to go.


http://reviews.llvm.org/D16530





More information about the llvm-commits mailing list