On Tue, Jan 12, 2016 at 12:30:20PM +0000, James Greenhalgh wrote: > > 2015-04-08 Jakub Jelinek <ja...@redhat.com> > > > > PR target/65689 > > * genpreds.c (struct constraint_data): Add maybe_allows_reg and > > maybe_allows_mem bitfields. > > (maybe_allows_none_start, maybe_allows_none_end, > > maybe_allows_reg_start, maybe_allows_reg_end, maybe_allows_mem_start, > > maybe_allows_mem_end): New variables. > > (compute_maybe_allows): New function. > > (add_constraint): Use it to initialize maybe_allows_reg and > > maybe_allows_mem fields. > > (choose_enum_order): Sort the non-is_register/is_const_int/is_memory/ > > is_address constraints such that those that allow neither mem nor > > reg come first, then those that only allow reg but not mem, then > > those that only allow mem but not reg, then the rest. > > (write_allows_reg_mem_function): New function. > > (write_tm_preds_h): Call it. > > * stmt.c (parse_output_constraint, parse_input_constraint): Use > > the generated insn_extra_constraint_allows_reg_mem function > > instead of always setting *allows_reg = true; *allows_mem = true; > > for unknown extra constraints. > > Hi Jakub, > > This applies clean to gcc-5-branch. I've bootstrapped and tested it on > x86_64-none-linux-gnu, aarch64-none-linux-gnu and arm-none-linux-gnueabihf > with no problems. > > Is this OK to commit to gcc-5-branch so I can close out PR 65689?
Ok, thanks. Jakub