[PATCH] D137811: InstCombine: Port amdgcn.class intrinsic combines to is.fpclass
Serge Pavlov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Nov 13 23:43:24 PST 2022
sepavloff added inline comments.
================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:824
+ // Equivalent of isnan. Replace with standard fcmp.
+ Value *FCmp = Builder.CreateFCmpUNO(Src0, Src0);
+ FCmp->takeName(&II);
----------------
Is it profitable to make such replacement early? Is there any advantage, at least hypothetical?
================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:846-850
+#if 0
+ if (isa<UndefValue>(Src0)) {
+ return replaceInstUsesWith(II, UndefValue::get(II.getType()));
+ }
+#endif
----------------
Deleted code?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137811/new/
https://reviews.llvm.org/D137811
More information about the llvm-commits
mailing list