[PATCH] D125670: [LSR] Early exit for RateFormula when it is already losing

Yueh-Ting (eop) Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 15:03:25 PDT 2022


eopXD added a comment.

In D125670#3542657 <https://reviews.llvm.org/D125670#3542657>, @Meinersbur wrote:

> Can you already trigger the `isValid` with the current code or is intended for you future changes? If the former, is it possible to add a test case? If the latter, could you highlight that in the summery (maybe even add "NFC" in the title)?

The current code does not trigger `isValid`. Yes, this patch should be an NFC.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125670



More information about the llvm-commits mailing list