[PATCH] D51406: [MC] fix a clang-tidy warning, NFC

Krasimir Georgiev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 29 01:40:11 PDT 2018


krasimir created this revision.
krasimir added a reviewer: bkramer.
Herald added a subscriber: llvm-commits.

Per clang-tidy:
function 'llvm::MCStreamer::checkCVLocSection' has a definition with different parameter names

.../llvm/lib/MC/MCStreamer.cpp:275:18: the definition seen here

.../llvm/include/llvm/MC/MCStreamer.h:235:8: differing parameters are named here: ('FuncId'), in definition: ('FunctionId')


Repository:
  rL LLVM

https://reviews.llvm.org/D51406

Files:
  lib/MC/MCStreamer.cpp


Index: lib/MC/MCStreamer.cpp
===================================================================
--- lib/MC/MCStreamer.cpp
+++ lib/MC/MCStreamer.cpp
@@ -272,10 +272,10 @@
                                     bool PrologueEnd, bool IsStmt,
                                     StringRef FileName, SMLoc Loc) {}
 
-bool MCStreamer::checkCVLocSection(unsigned FunctionId, unsigned FileNo,
+bool MCStreamer::checkCVLocSection(unsigned FuncId, unsigned FileNo,
                                    SMLoc Loc) {
   CodeViewContext &CVC = getContext().getCVContext();
-  MCCVFunctionInfo *FI = CVC.getCVFunctionInfo(FunctionId);
+  MCCVFunctionInfo *FI = CVC.getCVFunctionInfo(FuncId);
   if (!FI) {
     getContext().reportError(
         Loc, "function id not introduced by .cv_func_id or .cv_inline_site_id");


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51406.163026.patch
Type: text/x-patch
Size: 805 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180829/f0e82f11/attachment.bin>


More information about the llvm-commits mailing list