On Sun, Dec 09, 2018 at 03:04:02AM +0000, Ramsay Jones wrote: > On 08/12/2018 22:04, Luc Van Oostenryck wrote: > > @@ -924,6 +924,19 @@ static int handle_fpasses(const char *arg, const char *opt, const struct flag *f > > + char *buf = (char *) opt; > > + char *name; > > + int i; > > + > > + for (i = 0; i < max && (name = strsep(&buf, ",")); i++) > > Hmm, i = 0, so the first address-space name is given the > 'invalid' address-space number? It was more for consistency reasons. In the kernel the address space 1 & 2 are called '__user' & '__iomem' and, in the few place where it is needed, the default AS (0) is called '__kernel'. The idea here was to be able to specify them all: -faddress-space-names=__kernel,__user,__iomem, ... but '__kernel' would never be used (unless some debug code ask for it). Anyway, this patch will be dropped with the next version. -- Luc
