azabaznov added a comment.

> Anyway since there is not clear benefit that can be found now for the pragma 
> I think we should minimize its use as much as possible.

Unfortunately it's already there :(

> Anyway since there is not clear benefit that can be found now for the pragma 
> I think we should minimize its use as much as possible.

I personally agree, but I believe in order to go forward this patch should 
introduce diagnostics to commit the introduction of new functionality, I'm not 
sure what exactly it should be. I'm thinking of some pedantic one such as: 
"pragma enable is no longer required for type usage". What do you think?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100980/new/

https://reviews.llvm.org/D100980

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to