[PATCH] D159256: [NFC][Clang] Remove redundant function definitions

[Juan Manuel Martinez Caamaño via Phabricator via cfe-commits]

jmmartinez added a comment.

In D159256#4630876 <https://reviews.llvm.org/D159256#4630876>, @jhuber6 wrote:

> In D159256#4630410 <https://reviews.llvm.org/D159256#4630410>, @jmmartinez wrote:
>
>> @jhuber6 I was wondering if there is a reason you kept 3 versions of `mergeDefaultFunctionDefinitionAttributes` in https://reviews.llvm.org/D152391 ?
>
> I believe it's because one was a freestanding function, the other was a member function, and the last was a common implementation.

Would it be ok if I keep only one? It seems that the member function is not used (I was not sure if there was some external code using it).

If not, I can also keep just 2 versions (the freestanding function and the member function), move the implementation to the freestanding one, and drop the static function since it is redundant.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D159256/new/

https://reviews.llvm.org/D159256