[llvm] [GlobalISel] Document minimum legality requirements for G_IMPLICIT_DEF. (PR #117609)

Amara Emerson via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 25 13:51:31 PST 2024


aemerson wrote:

> I don't see any benefit of this change. It makes the state worth. It gives you an escape-hatch that you don't have to query the legalizer in some situations.

That's the entire point of this change. It's pointless to have to query for legality and slow down compilation when LLVM codegen fundamentally breaks when some constructs are not legal.

> All non-Apple:-)

Ok, I was asking to give you the benefit of the doubt. Having clarified that you meant check downstream targets that are private to myself, you're now admitting that your statement before was entirely sarcastic without any purpose in the discussion?



https://github.com/llvm/llvm-project/pull/117609


More information about the llvm-commits mailing list