ssahasra wrote: > Thanks for the feedback. I am pretty new to working with tablegen, so I'm not > sure what the right approach would be. The previous version was based on how > `AlwaysUniform` was implemented. Let me know if this new version is better.
This version is certainly promising. I can see the value of having an auto-generated lookup table, so that we don't add any actual field to the Intrinsic class. But I am really not experienced enough to evaluate this change. I would rather watch and learn myself. https://github.com/llvm/llvm-project/pull/225488 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
