On Thu, Jan 30, 2014 at 4:45 AM, Jason Merrill <ja...@redhat.com> wrote:
> If a parameter of a covariant virtual function is volatile, we can't just
> use the parameter directly in the call we build for the thunk, or the gimple
> verifier will complain.  We need to copy it into a temporary first.
>
> Tested x86_64-pc-linux-gnu.  OK for trunk?

Please use

  if (!is_gimple_val (arg))

instead of testing TREE_THIS_VOLATILE, then it's also obvious that
you can remove the assert (I suppose it would trip for an argument
of volatile struct X for example?).

Ok with that change.

Thanks,
Richard.

Reply via email to