benlangmuir wrote:

> Ok, so what do you suggest? Change all CODEGENOPT to BENIGN_CODEGENOPT before 
> this PR lands?

This would be fine with me.

> (Or alternatively/equivalently go all the way and add an explicit benign 
> effect on AST argument to CODEGENOPT, similar to what I linked above for 
> LangOptions?) 

This would also be fine with me. I don't have a strong preference either way.



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

Reply via email to