[PATCH] D31211: [AMDGPU] Add target feature new-addr
Yaxun Liu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 21 15:14:36 PDT 2017
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
More information about the llvm-commits
mailing list