[PATCH] D21183: Better selection of common base address in constant hoisting

Juergen Ributzka via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 10 11:01:30 PDT 2016


ributzka added a comment.

Mhh. The fact that AArch64 doesn't even implement the TTI hook and shows different behavior is not great - especially since it seems to have increased code size.

In its current form this is still a very targeted optimization for Thumb1 load/store. Wouldn't an ARM specific MachineFunction pass be better for this?


http://reviews.llvm.org/D21183





More information about the llvm-commits mailing list