[PATCH] D122608: Fix behavior of ifuncs with 'used' extern "C" static functions

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

FWIW, I'm seeing a precommit CI failure on Windows:

  Failed Tests (1):
    Clang :: SemaCXX/externc-ifunc-resolver.cpp

May as well also fix up the clang-format issues in the review.



================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:6319
+bool CodeGenModule::CheckAndReplaceExternCIFuncs(
+    llvm::GlobalValue *Elem, IdentifierInfo *Name,
+    llvm::GlobalValue *CppFunc) {
----------------
`Name` appears to be entirely unused?


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

https://reviews.llvm.org/D122608