Hi Emese, 2016-05-03 2:56 GMT+09:00 Emese Revfy <re.emese@xxxxxxxxx>: > On Mon, 2 May 2016 14:03:00 +0900 > Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> wrote: > >> In the first place, >> I am wondering if we need to revive this documentation. >> What this commit is only interested in *.so generation, >> not host program support. > > I agree that we don't need this documentation. I'll remove it. > >> > @@ -124,5 +158,39 @@ quiet_cmd_host-cxxobjs = HOSTCXX $@ >> > $(host-cxxobjs): $(obj)/%.o: $(src)/%.cc FORCE >> > $(call if_changed_dep,host-cxxobjs) >> > >> > +# Compile .c file, create position independent .o file >> > +# host-cshobjs -> .o >> > +quiet_cmd_host-cshobjs = HOSTCC -fPIC $@ >> > + cmd_host-cshobjs = $(HOSTCC) $(hostc_flags) -fPIC -c -o $@ $< >> > +$(host-cshobjs): $(obj)/%.o: $(src)/%.c FORCE >> > + $(call if_changed_dep,host-cshobjs) >> > + >> > +# Compile .c file, create position independent .o file >> >> >> Please explain why c++ compiler should be used to compile .c files. >> >> This is not clear, so worth commenting. > > Where do you think that the C++ compiler is used above? I think HOSTCC can only be C compiler. > Not above, but *below*. This comment "Compile .c file, create position independent .o file" is for the following code block. # Compile .c file, create position independent .o file # host-cxxshobjs -> .o quiet_cmd_host-cxxshobjs = HOSTCXX -fPIC $@ cmd_host-cxxshobjs = $(HOSTCXX) $(hostcxx_flags) -fPIC -c -o $@ $< $(host-cxxshobjs): $(obj)/%.o: $(src)/%.c FORCE $(call if_changed_dep,host-cxxshobjs) We generally use HOSTCC to compile *.c files, and HOSTCXX to compile *.cc files. But, here, you mention to use HOSTCXX to compile .c files such as cyc_complexity_plugin.c, sancov_plugin.c, etc. This is not straight-forward. It is worthwhile to comment the reason. -- Best Regards Masahiro Yamada -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html