[PATCH] D41652: [InstCombine] Add an option to disable addrspacecast folding into GEP

Marek Olšák via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 2 06:19:40 PST 2018

mareko added a comment.

In https://reviews.llvm.org/D41652#965765, @davide wrote:

> I'm not sure having a `cl::opt` is the best option here.
>  If you really want to make such a change, can you at least thread this information through `TargetTransformInfo` rather than using a global option?


> Also, for my own curiosity, is this a temporary workaround until AMDGPU grows proper support for 32-bit GEPs or is this an inherent limitation?
>  If the former, maybe this hack should not go in (or at least we should consider what's the amount of work needed to finish implementing the support)

It's possible that this will be a permanent solution, because we don't plan to have full 32-bit support (it would be too much work in the backend for little benefit).


More information about the llvm-commits mailing list