arsenm added inline comments.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:16212
+ case AMDGPU::BI__builtin_amdgcn_flat_atomic_fmax_f64: {
+ Intrinsic::ID IID;
+ llvm::Type *ArgTy = llvm::Type::getDoubleTy(getLLVMContext());
----------------
rampitec wrote:
> arsenm wrote:
> > rampitec wrote:
> > > gandhi21299 wrote:
> > > > rampitec wrote:
> > > > > You do not need any of that code. You can directly map a builtin to
> > > > > intrinsic in the IntrinsicsAMDGPU.td.
> > > > Sorry, I looked around for several days but I could not figure this
> > > > out. Is there a concrete example?
> > > Every instantiation of `GCCBuiltin` in the `IntrinsicsAMDGPU.td`.
> > This is not true if the intrinsic requires type mangling. GCCBuiltin is too
> > simple to handle it
> Yes, but these do not need it. All of these builtins are specific.
These intrinsics are all mangled based on the FP type
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106909/new/
https://reviews.llvm.org/D106909
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
