================
@@ -7151,6 +7151,561 @@ void Verifier::visitIntrinsicCall(Intrinsic::ID ID, 
CallBase &Call) {
           &Call, Op);
     break;
   }
+  case Intrinsic::amdgcn_image_load_1d:
----------------
AlexeySachkov wrote:

This list is enormously huge which I don't like and I'm looking for feedback 
from reviewers:
- there is a searchable table for those intrinsics which we can use to detect 
them, but that will mean a runtime lookup instead of `switch...case`
- we could generate some `.def`/`.inc` file to outline all those `case 
Intrinsic::...` in there
- we could create a separate function (or even file) to outline all `amdgcn` 
intrinsics check in there so that they do not pollute the existing 
`switch...case` as much

https://github.com/llvm/llvm-project/pull/179511
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to