[PATCH] D67404: [ARM] LE support in ConstantIslands

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 17 00:27:34 PDT 2019


samparker added a comment.

I'd say that it's sub-optimal... I don't know how often the issue will arise and whether that justifies using LoopInfo here. I'm also unsure how that extra logic would fit in with the current optimisation as we'd want to visit inner loops first, instead of walking backwards.


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

https://reviews.llvm.org/D67404





More information about the llvm-commits mailing list