[PATCH] D88953: [LAA] Use DL to get element size for bound computation.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 10:18:54 PDT 2020


fhahn added a comment.

In D88953#2317212 <https://reviews.llvm.org/D88953#2317212>, @efriedma wrote:

> On a related note, the amount of code we generate in runtime checks for the given testcase is completely ridiculous. But that's not a correctness issue. :)

Agreed, but I think this mostly boils down to improving SCEV expansion and follow-up simplifications, as LAA mostly just hands off SCEV expression to the expander.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88953



More information about the llvm-commits mailing list