This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [COMMITTED] Fix silly error in aarch64_register_move_cost


On 04/18/14 10:02, Richard Henderson wrote:
Building mainline I got

.../aarch64.c:4879:134: error: invalid conversion from âreg_class_t {aka int}â to âmachine_modeâ [-fpermissive]
    if (! TARGET_SIMD && GET_MODE_SIZE (from) == 128 && GET_MODE_SIZE (to) == 128)

Sure enough, TO and FROM are not modes.  Did mainline just change away from
permissive or something?  It surely seems like we ought to have seen this
earlier...
I haven't looked at GET_MODE_XXX, but most, if not of the RTL checking is disabled because it's too expensive.

I'm hoping that David's work will take us to a place where we can do more static checking on the RTL bits too.

jeff


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]