This finishes off a year long pursuit to LLVMify the generated
operation accessors, prefixing them with get/set. Support for
any other accessor naming is fully removed after this commit.
https://discourse.llvm.org/t/psa-raw-accessors-are-being-removed/65629
Is this recorded somewhere still post? This is default now.
We may also want to warn when attributes overlap with more helpful message in the case where it is due to this transform, e.g., t and T now map to same accessor where it didn't before and some renaming would be needed (a bit unfortunate that I didn't add that check earlier). [Not blocker, could just be GitHub issue too]