[PATCH] D56135: Allow reverse iterators for `MachineBasicBlock::findDebugLoc` and `MachineBasicBlock::findPrevDebugLoc`

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 11 08:39:46 PST 2019


fhahn added a comment.

In D56135#1347626 <https://reviews.llvm.org/D56135#1347626>, @m4yers wrote:

> In D56135#1343674 <https://reviews.llvm.org/D56135#1343674>, @fhahn wrote:
>
> > Are there any instances in the current codebase where this would be useful?
>
>
> For example `X86InstrInfo::AnalyzeBranchImpl` can be rewritten with more natural reverse iterators.


Would it be possible to post that as a follow up patch?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56135





More information about the llvm-commits mailing list