[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