[PATCH] D151349: [HIP] emit macro `__HIP_NO_IMAGE_SUPPORT`
Yaxun Liu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 30 10:17:38 PDT 2023
yaxunl added a comment.
In D151349#4381471 <https://reviews.llvm.org/D151349#4381471>, @arsenm wrote:
> In D151349#4377792 <https://reviews.llvm.org/D151349#4377792>, @yaxunl wrote:
>
>> using ISA version to determine whether image is supported
>
> That’s backward. You can track the feature in clang separately and just not emit it. We do that for a few other features. You just stop the TargetParser parts
Sorry, I did not quite get it. Do you mean to add a feature to https://github.com/llvm/llvm-project/blob/85670ac86813b170c9301aa477421c56a71a7e1e/llvm/lib/TargetParser/TargetParser.cpp#L107 ? should I add FEATURE_IMAGE or FEATURE_NO_IMAGE ?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151349/new/
https://reviews.llvm.org/D151349
More information about the llvm-commits
mailing list