On Tue, Jun 13, 2017 at 8:32 AM, H.J. Lu wrote: > On Tue, Jun 13, 2017 at 4:54 AM, H.J. Lu wrote: >> On Tue, Jun 13, 2017 at 4:46 AM, Andreas Schwab wrote: >>> On Jun 13 2017, "H.J. Lu" wrote: >>> >>>> The first / in "/$1:/{s/:\$//;p}" is a command: >>> >>> It's not a command, it's an address. >>> >>>> [hjl@gnu-tools-1 glibc-test]$ echo "#include " | >>>> /usr/gcc-7.1.1-x32/bin/c++ -M -MP -x c++ - | sed -n >>>> ",/bits/std_abs.h:,{s/:\$//;p}" >>>> sed: -e expression #1, char 1: unknown command: `,' >>>> >>>> What options do I have? >>> >>> \,$1, >> >> Here is the updated patch. OK for master? >> >> Thanks. > > I am checking it in. > A new target for $(common-objpfx)bits/std_abs.h is needed. Tested and verified with GCC5, GCC 6 and GCC 7 on x86-64. This is what I am checking in. -- H.J.