[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