[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