================
@@ -657,7 +657,7 @@ MCPseudoProbeDecoder::getCallProbeForAddr(uint64_t Address) 
const {
   const auto &Probes = It->second;
 
   const MCDecodedPseudoProbe *CallProbe = nullptr;
-  for (const auto &Probe : Probes) {
----------------
aaupov wrote:

> Ok, then this change should be part of #102789, and should not be separated. 
> Otherwise the motivation and effect of this change is lost, because the base 
> side of this change still use `std::list`, and the change description stating 
> "trigger type casts" can be confusing.
> 
> Suggest you merge it back (or at least fix change description).

I intended to minimize the changes in #102789 and make all preparation changes 
small, isolated, and verifiably NFC. I've changed the description, hope it 
makes sense now.

https://github.com/llvm/llvm-project/pull/102788
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to