https://github.com/nikic commented:

I think the general direction is fine here. I would suggest to split this up 
into three parts: 1. Add noext attribute and disabled verification, 2. fix any 
verification failures (e.g. by emitting noext), 3. enable verification.

It would be good to still be able to enable the verification in llc via an opt 
option, so you can write tests for it (and maybe useful when reducing an 
issue). 

https://github.com/llvm/llvm-project/pull/100757
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to