[PATCH] D92246: [DAGCombiner][NFC] Replace duplicate implementation flipBoolean with DAG.getLogicalNOT
Layton Kifer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 27 13:42:07 PST 2020
laytonio added a comment.
I'm not sure I understand the concern here. If its just the requirement to specify a VT, maybe we should add an overload of getLogicalNOT that doesn't require one? (and maybe getNOT while we're at it) It seems reusing the VT of whatever you're flipping is probably what you want most of the time anyway.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the llvm-commits