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

Joseph Huber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 31 04:56:31 PDT 2023


jhuber6 added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159256



More information about the cfe-commits mailing list