[PATCH] D104282: [FuncSpec] Use std::pow instead of operator^

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 15 01:15:42 PDT 2021


ChuanqiXu added inline comments.


================
Comment at: llvm/test/Transforms/FunctionSpecialization/function-specialization-loop.ll:2
+; RUN: opt -function-specialization -func-specialization-avg-iters-cost=3 -S < %s | FileCheck %s
+
+; check the bonus for specializing constant arguments are powered by the depth of the loop.
----------------
ChuanqiXu wrote:
> SjoerdMeijer wrote:
> > Just checking, this test didn't trigger before, and now it does? 
> > If so, would be good to precommit the test, then we can also see the differences better here.
> Yeah, I had checked that.
Do you mean that send a NFC patch which contains the test case only? Then we tried to update this one, is it?


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

https://reviews.llvm.org/D104282



More information about the llvm-commits mailing list