[PATCH] D46147: [NVPTX] Added a feature to use short pointers for const/local/shared AS.

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In https://reviews.llvm.org/D46147#1081951, @bkramer wrote:

> Would it be possible to get rid of the flag entirely and make this controlled only by the data layout? That's defined on the module level.


Yes, ultimately the switch is just changing the datalayout. If this is just a short term thing for migration I think a global flag is fine. If you want to be able to swap these properly long term, this is a different arch name / TargetMachine.


https://reviews.llvm.org/D46147