Explain IR acronym in README #7506
Conversation
README.md
Outdated
| @@ -5,7 +5,7 @@ | |||
|
|
|||
| # ⚡️ "CIRCT" / Circuit IR Compilers and Tools | |||
|
|
|||
| "CIRCT" stands for "Circuit IR Compilers and Tools". One might also interpret | |||
| "CIRCT" stands for "Circuit Intermediate Representation (IR) Compilers and Tools". One might also interpret | |||
There was a problem hiding this comment.
I think we need to discuss whether in the title this refers to Intermediate Representation or Intermediate Representations. I personally opt for the plural form as the various dialects function more as a set of inter-operable IRs rather than a single IR.
| "CIRCT" stands for "Circuit Intermediate Representation (IR) Compilers and Tools". One might also interpret | |
| "CIRCT" stands for "Circuit Intermediate Representations (IR) Compilers and Tools". One might also interpret |
There was a problem hiding this comment.
Will wait for one more person to agree before I make the change, unless you think I should just do it now!
I personally prefer singular, but am not very familiar with this project
There was a problem hiding this comment.
It also makes sense because the rest of the title is in plural form as well. Also as I explained there is more than one IR in the project and the issue you mentioned wasn't an answer to "what does IR refer to in the title" but rather "what does IR mean in general", which is a very commonly used acronym in the field. So I definitely think that it should be plural.
|
Pluralized! |
Fixes #7323. Change is pre-approved.