[PATCH] D71857: [NFC] Fixes -Wrange-loop-analysis warnings

Mark de Wever via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 1 07:26:28 PST 2020


Mordante marked 8 inline comments as done.
Mordante added inline comments.


================
Comment at: lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptx86ABIFixups.cpp:22
 
 #include "lldb/Target/Process.h"
 #include "lldb/Utility/Log.h"
----------------
xbolva00 wrote:
> NFC patch?
Changed to NFC.


================
Comment at: lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptx86ABIFixups.cpp:75
     return false;
-  for (const auto &param : call_inst->operand_values())
+  for (auto param : call_inst->operand_values())
     if (isRSAllocationPtrTy(param->getType()))
----------------
aaron.ballman wrote:
> `auto *`?
Changed to  `const auto *`


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

https://reviews.llvm.org/D71857





More information about the llvm-commits mailing list