Rahul Joshi comments

Repositories
Issues
Comments

Results 3 comments of


                                            Rahul Joshi

[LLVM] Change Intrinsic::ID to encode target and intrinsic index

> > This change is in preparation for being able to disable intrinsics for targets that are not enabled. > > Why is it required for that? Is there some...

[LLVM] Change Intrinsic::ID to encode target and intrinsic index

> Thanks, I'm happy with this, but please make sure other reviewer comments are addressed. Sure. However, I think there is an open discussion on discourse that needs resolution to...

[LLVM] Change Intrinsic::ID to encode target and intrinsic index

> > Have you checked whether this does help incremental builds in practice? > > I checked and it does seem to help. Adding a new AMDGPU intrinsic did not...