[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