ssahasra wrote:

This is an unfamiliar use of TableGen for me, and I haven't used TableGen 
itself in quite some time, so my knowledge is outdated. But there is a code 
smell. Any scheme that needs to repeat the name of a def in a TD file seems 
like bad design. There should be one place in the intrinsic def itself where it 
should be possible to say "this is an async intrinsic". It could be a property 
in the record, or it could be a class that the def derives from.

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

Reply via email to