[PATCH] D127460: Rename GCCBuiltin into ClangBuiltin
Guillaume Gomez via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 22 05:30:29 PDT 2022
GuillaumeGomez added a comment.
In D127460#3601193 <https://reviews.llvm.org/D127460#3601193>, @xbolva00 wrote:
> Patch description needs to be added, especially answer the question why we should rename it, why we want it..
I updated the description to explain why it is needed. I'm not sure how big it's supposed to be so I didn't enter into the details though.
In D127460#3601205 <https://reviews.llvm.org/D127460#3601205>, @xbolva00 wrote:
> In D127460#3601152 <https://reviews.llvm.org/D127460#3601152>, @GuillaumeGomez wrote:
>> This is my first LLVM contribution so I don't think I do? If I do have a commit access anyway, do you have a link to the documentation where it explains what I'm supposed to do by any chance?
Thanks! Once this is approved, I'll do it then.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits