[PATCH] Introducing compiler plugin support
Michael Stahl
mstahl at redhat.com
Mon Oct 8 08:01:33 PDT 2012
On 05/10/12 18:22, Lubos Lunak wrote:
>
> Hello,
>
> attached is my implementation of basic build support for using Clang compiler
> plugin during building LO. I'm posting it here first in case our build system
> people have different ideas to how I have integrated it in the build system,
> if there are no comments, I will push it.
hmm... so that effectively needs to be built as a "bootstrap" step and
the build system basically assumes that it's been done already. which
seems appropriate since probably ~everything would need to have a
dependency on it otherwise.
the stuff used for bootstrap seems to live in the top-level directory
and solenv/ (and dmake/) currently... but i don't really have an opinion
on how that should look...
only problem is that this seems to generate files below the new
compilerplugins/ dir, in the source tree... which doesn't make it any
worse than other parts of the bootstrapping.
> +$(CLANGOUTDIR)/compileplugin.so: $(2)
> +$(CLANGOUTDIR)/compileplugin.so: CLANGOBJS += $(2)
> +
> +$(CLANGOUTDIR)/compileplugin.so: $(CLANGOBJS)
there's a bit of redundancy there.
hmm actually the .so only has .o dependecies so could just use $^ in the
command.
> +$(CLANGOUTDIR):
> + mkdir -p $(CLANGOUTDIR)
i vaguely remember problems with directories in make 3.81 but that was
probably with pattern rules... ah right in a trivial test this seems to
work with 3.81.
More information about the LibreOffice
mailing list