HazardyKnusperkeks wrote:

Smaller changes are easier to review, that is obvious and I support it.
For what do we have the `ShortBlocks` anyway? If `If` and `Loops`, enums, 
cases, etc. are handled separately, what is left?

@rudolflovrencic I have no problem with adding the options. But when you want 
to invest some time in the area can I make a small request? I have always had a 
problem with the name `Allow...` because it doesn't allow, it forces either 
way. Allow would in my eyes be `Leave` like we have in other options. There 
would be 2 ways to handle it, either keep the name and add a `Leave` option, or 
change the name (and then it's still open to add `Leave`).

https://github.com/llvm/llvm-project/pull/196021
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to