[PATCH] D31211: [AMDGPU] Add target feature new-addr

[Yaxun Liu via Phabricator via llvm-commits]

yaxunl added a comment.

In https://reviews.llvm.org/D31211#706892, @arsenm wrote:

> This can't be a subtarget feature. This needs to be a global option since this is a property of the module


By convention, Clang use -m options for target specific options, e.g. -mmmx. Clang translates them to target feature string by dropping -m. When TargetInfo is constructed in Clang, only TargetOption is available. I don't see a better way to define a Clang option which is only used by AMDGPU target.


https://reviews.llvm.org/D31211