[Lldb-commits] [PATCH] D71857: Fixes -Wrange-loop-analysis warnings

2020-01-01 Thread Dávid Bolvanský via Phabricator via lldb-commits
xbolva00 added inline comments. Comment at: lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptx86ABIFixups.cpp:22 #include "lldb/Target/Process.h" #include "lldb/Utility/Log.h" NFC patch? Repository: rG LLVM Github Monorepo

[Lldb-commits] [PATCH] D71857: Fixes -Wrange-loop-analysis warnings

2020-01-01 Thread Aaron Ballman via Phabricator via lldb-commits
aaron.ballman added a comment. It seems like some of the deductions are not spelling out the qualifiers or pointer/references explicitly in some cases, at least from my spot-checking. Can you go back through this patch to make sure we're getting those correct? Comment at: cla

[Lldb-commits] [PATCH] D71857: Fixes -Wrange-loop-analysis warnings

2019-12-24 Thread pre-merge checks [bot] via Phabricator via lldb-commits
merge_guards_bot added a comment. Herald added a subscriber: wuzish. {icon check-circle color=green} Unit tests: pass. 61116 tests passed, 0 failed and 728 were skipped. {icon times-circle color=red} clang-tidy: fail. Please fix clang-tidy findings

[Lldb-commits] [PATCH] D71857: Fixes -Wrange-loop-analysis warnings

2019-12-24 Thread Mark de Wever via Phabricator via lldb-commits
Mordante created this revision. Mordante added reviewers: aaron.ballman, xbolva00. Mordante added projects: LLVM, LLDB, clang. Herald added subscribers: bmahjour, usaxena95, kadircet, arphaman, jkorous, kbarton, hiraditya, nemanjai, qcolombet, MatzeB. This avoids new warnings due to D68912