[PATCH] D109676: [HardwareLoops] put +1 for loop count before zero extension

Thu Dec 2 20:01:06 PST 2021

shchenz marked an inline comment as done.
shchenz added a comment.

> So for this patch, we need to wait for ARM experts find out the solution for the overflow issue when ExitCount type is same with CountType?

Not sure if there is any solution for the above issue? @samparker @dmgreen .

This patch is required by the PowerPC as it can generate better instructions shown in the test changes. And this should not introduce any regression on ARM, because the codes before this patch also have the potential issue. See https://reviews.llvm.org/D109676#3077240

So, is it possible to get this in first? Thanks in advance.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109676