Breaking a line after `DECL_P (` in the relocated `validate_attr_args` is not the normal style used. Other than that reformatting, the move of C front-end code is OK. Note that I have not reviewed functions relocated from the JIT front end such as `handle_transaction_pure_attribute`.
-- https://forge.sourceware.org/gcc/gcc/pulls/207#issuecomment-7560
