Pushed on your behalf. Welcome to GCC! :)
> Hi, > > This is v2 of the patch. > > Changes since v1: Fix indentation after removal of the conditional > statements, as suggested in review. > > This patch removes redundant caller-side HAS_ROWS checks before calls to > a68_low_dup. > > a68_low_dup originally always performed a deep copy, so callers avoided > invoking it for values without rows by checking HAS_ROWS beforehand. > > a68_low_dup now performs its own HAS_ROWS check and returns the original > expression unchanged when a deep copy is not required. Remove the > remaining caller-side HAS_ROWS checks, as they are redundant. > > Thanks, > Kanishka Solanki
