[PATCH] D118642: [IVDescriptor] Find original 'Previous' for first-order recurrences.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 27 04:57:18 PST 2022


fhahn added a comment.

In D118642#3337470 <https://reviews.llvm.org/D118642#3337470>, @spatel wrote:

> I am not familiar with this code, so adding some likely better reviewers.
>
> The description suggests that we have a potential to miscompile. Is that shown in the test diffs?

Thanks. This shouldn't fix a mis-compile, it just handles more cases by replacing the exit `return Previous->comesBefore(LastPrev);` to handle additional cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118642



More information about the llvm-commits mailing list