On Sun, Mar 14, 2021 at 12:43 PM Uros Bizjak <ubiz...@gmail.com> wrote: > > On Sun, Mar 14, 2021 at 8:14 PM H.J. Lu <hjl.to...@gmail.com> wrote: > > > > Done. Here is the updated patch. Tested on Linux/x86-64. OK for > > > > master? > > > > > > I don't understand the purpose of the current_output_insn check and I > > > don't know if the usage of current_output_insn is correct. The > > > comments are not helpful either, and no other target uses this > > > variable in the way you propose. Can you please elaborate the reason > > > and the purpose of the check a bit more? > > > > > > Uros. > > > > Originally, ix86_force_load_from_GOT_p is only for non-PIC. My patch > > extended > > it to inline assembly statements where current_output_insn == NULL and PIC > > is > > allowed in 64-bit. > > I can see this from the patch, but this explanation didn't answer my question. >
The purpose of current_output_insn == NULL is to allow PIC for inline asm statements in 64-bit mode. Is there a better way to check if ix86_print_operand () is called on inline asm statements? -- H.J.
