[PATCH] D110322: [ConstantFolding] Fold constrained compare intrinsics
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Dec 18 23:23:43 PST 2021
craig.topper added inline comments.
================
Comment at: llvm/lib/IR/ConstantFold.cpp:2021
+/// Evaluate the given predicate for the constant values.
+bool llvm::evaluatePredicate(unsigned Pred, const APFloat &Op1,
+ const APFloat &Op2) {
----------------
spatel wrote:
> sepavloff wrote:
> > spatel wrote:
> > > This was suggested in an earlier comment - can we add this next to:
> > > https://github.com/llvm/llvm-project/blob/80b1d472d6f7e10403e37cc74bfda278372ecb97/llvm/lib/IR/Instructions.cpp#L4135
> > >
> > > (and adjust the names/parameters for consistency)
> > >
> > > That can be a preliminary/NFC commit if I'm seeing it correctly.
> > Previously such method existed but I was recommended to remove it: https://reviews.llvm.org/D110322?id=388475#3159399.
> >
> > Otherwise there is no problem to restore that implementation.
> Ah, sorry I didn't see that earlier comment/revision.
> @craig.topper - do you still prefer the current structure?
I think the integer change happened while this patch was in review. I'm fine with restoring the previous implementation to make this consistent with integer.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110322/new/
https://reviews.llvm.org/D110322
More information about the llvm-commits
mailing list