[PATCH] D84694: [NewPM] Rename function-attrs -> functionattrs
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 27 12:21:54 PDT 2020
aeubanks added a comment.
In D84694#2176745 <https://reviews.llvm.org/D84694#2176745>, @arsenm wrote:
> I think this is the backwards direction. I think most passes consistently use - as a word separator, and the annoying cases do not
Sure I can go the other direction, I went this way because it was more consistent with relevant passes, but I agree that a dash is better overall.
Looks like I'd only have to update 43 tests for "functionattrs" -> "function-attrs".
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84694/new/
https://reviews.llvm.org/D84694
More information about the llvm-commits
mailing list