[PATCH] D116778: [clang-tidy][clang] Don't trigger unused-parameter warnings on naked functions
Tommaso Bonvicini via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 27 11:08:14 PST 2022
MuAlphaOmegaEpsilon added a comment.
In D116778#3276421 <https://reviews.llvm.org/D116778#3276421>, @aaron.ballman wrote:
> In D116778#3259189 <https://reviews.llvm.org/D116778#3259189>, @MuAlphaOmegaEpsilon wrote:
>
>> In D116778#3255414 <https://reviews.llvm.org/D116778#3255414>, @aaron.ballman wrote:
>>
>>> LGTM! The CI failure is finally down to just an unrelated one (yay?). Do you need someone to commit on your behalf? If so, what name and email address would you like used for patch attribution?
>>
>> Great! 🎉
>> I've never committed on the LLVM repository, so I don't know the requirements. If it doesn't bother you, feel free to commit on my behalf using the following:
>> MuAlphaOmegaEpsilon <tommasobonvicini at gmail.com>
>>
>> Thank you very much Aaron! 😀
>
> Sorry about the delay in getting this landed (it slipped beneath my radar). I've commit on your behalf in ccce1a03c9ce9c3917b310097c89e39bb68527e2 <https://reviews.llvm.org/rGccce1a03c9ce9c3917b310097c89e39bb68527e2>. Thank you for the fix!
No problem, I wasn't in a hurry! :)
Thank you very much again Aaron! 😀
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116778/new/
https://reviews.llvm.org/D116778
More information about the cfe-commits
mailing list