arsenm wrote:

> > I'd argue that we could simply add IsLDSDMA to the searchable tables over 
> > in `AMDGPUSearchableTables.td`? We've already got SourceOfDivergence there, 
> > for example.
> 
> I don't understand that design at all. My immediate reaction is that 
> repeating the name of a def in a different place smells of bad design. What 
> am I missing?

Yes, that is annoying. Though I am more bothered by the fact that we specify 
this in the unsafe direction

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