[PATCH] D57044: [AArch64] Optimize Inf materialization
Adhemerval Zanella via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 23 10:05:18 PST 2019
zatrazz added a comment.
In D57044#1367943 <https://reviews.llvm.org/D57044#1367943>, @efriedma wrote:
> Is there some reason we're checking for specific floating-point constants here, as opposed to just calling AArch64_AM::isLogicalImmediate or AArch64_AM::isAnyMOVWMovAlias?
I am not sure, the DAGCombiner code seems to check on some points if the operation is legal and other if constant can be materialized without load (DAGCombiner::convertSelectOfFPConstantsToLoadOffset).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57044/new/
https://reviews.llvm.org/D57044
More information about the llvm-commits
mailing list