I intend to submit https://reviews.llvm.org/D90859 sometime early next week which will change the syntax of FuncOp to be
func "<visibility>"? @func_name ... and then drop
sym_visibility from the attribute dict. The old syntax will still continue to be accepted but the printer will always generate the new syntax. As such, if your (out-of-tree) tests CHECK old syntax, they will need to be updated to CHECK the new syntax.
Also note that if your code directly uses
parseFunctionLikeOp, they continue to support the existing behavior by default.
Please let me know if there are concerns.