[PATCH] D151349: [HIP] emit macro `__HIP_NO_IMAGE_SUPPORT`

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