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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 10:11:43 PDT 2020


efriedma added a comment.

The code change looks right to me, but it's my first time looking at this code.

I verified this fixes the original program I was looking at.

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. :)


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