Serosh-commits wrote:

> I believe the better fix would be on entry to `checkBuiltinTemplateIdType` -- 
> there's not really a good reason to not just make the fix universal.

@ojhunt 

> I believe the better fix would be on entry to `checkBuiltinTemplateIdType` -- 
> there's not really a good reason to not just make the fix universal.

good point  thanks for ur advice i moved the dependency check to the entry 
point so it’s universal now it definitely makes the code cleaner since i could 
remove the duplicated checks in the other builtins. let me know if this looks 
better to you.

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

Reply via email to