Age | Commit message (Collapse) | Author |
|
Also create a Clang compiler plugin to detect such cases.
Change-Id: I61ad1a1d6b1c017eeb51f226d2dde0e9bb7f1752
Reviewed-on: https://gerrit.libreoffice.org/4001
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
A different way to do 1c0669af2f1f58e6431b5e489ac48a883e242ba7.
Sometimes one piece of code can be represented several times in the AST,
e.g. with default function arguments.
Change-Id: Ic7799fa0bd918a638bdc8ebef69e6aa91d355bdc
|
|
Change-Id: Ibc487aa78d9eba3f936b4f4cc0298adfaef60d5c
|
|
This does not always work well, e.g. when building a return value
in a return statement from a temporary, there is CXXConstructExpr
containing CXXTemporaryObjectExpr, which both share the same location.
This reverts commit 1c0669af2f1f58e6431b5e489ac48a883e242ba7.
|
|
Change-Id: Icba8dfa61aee7237af569b2414b1cf4ce93ca476
|
|
Turns out removeText( SourceRange ) treats it as a token range, so it's
not always character-exact if used for removal of only several characters
from a token (e.g. an identifier).
Change-Id: I0223d52da90f9535d9ef1d48b0f56d69131536c8
|
|
Change-Id: I71236b9ca6300372ba00c85401cf19f6c0e7ac99
|
|
Change-Id: If54a3d7047f13ae9c9345c21737a89afee645403
|
|
Needs work to actually do something useful, but the basics are there.
Change-Id: I193922f2f5572760c8c20def0f9b830138f47fef
|
|
Change-Id: I3bccfa773448e17ac3c8ece6b99f85c6d399b0cf
|
|
http://lists.freedesktop.org/archives/libreoffice/2013-March/047769.html
Change-Id: Ibba9d3dcc87d6d6eee58ab3690a8f87339d00956
|
|
If the class is based on PPCallbacks too, Clang internals will delete
the instance.
Change-Id: I6dd83d800e6cca17eb0b5de23c8994f11c087fd5
|
|
Change-Id: I4397575e4665e8a257d0bd3e15245d996a48a1f4
|
|
Change-Id: I415ed25586408d7e7df9457f7c637a8c6d13d35d
|
|
It's possible to get the latter from the former, and the former
is useful for other things too (access to the preprocessor, for example).
Change-Id: I708d709129fd3a35bf7c63da4de09c2e696b382d
|
|
Change-Id: I246dcf1d2c099c510435d21f0dd47e3c098456f2
|
|
Change-Id: Id2572982ca899223b89016ee7482ccb960032805
|
|
Change-Id: I4849656b16c2a7103217870050b0de3ccb97af56
|
|
Change-Id: Iadacffaad832c6ff06757e8567e24f929f24a4c3
|
|
Change-Id: Ib08c5075034e0e16c5541f05425ef15d2cc3a6f6
Reviewed-on: https://gerrit.libreoffice.org/2045
Reviewed-by: Luboš Luňák <l.lunak@suse.cz>
Tested-by: Luboš Luňák <l.lunak@suse.cz>
|
|
Change-Id: I4e3e8f5ca5e5b5b59b1bd067281f90940dd893b1
|
|
Make sure / is appended, otherwise /foo might match /foobar/file .
Change-Id: I36469916b72b407c8f0c9c255099ee671039cf17
|
|
So running the compiler plugin can be done either with FORCE_COMPILE_ALL=1
for simplicity, running the plugin on all source files for the target,
or it's possible to use make's -W option (or touch files manually) to
run it only for selected files.
Change-Id: Ifa82bbe91dc1e179af9db8fbf99271bcf03a1d47
|
|
Otherwise there's an error message e.g. during a clean build.
Change-Id: I42efd08e014a4ebe25cfd1a497f622e5ea5c39ad
|
|
CLANGFORCE was the first in the makefile after
c6ffe17631cccf11fbe00479d2169116d494a7da and therefore the default
Change-Id: Id3b86a1ea2c27479067f7f55a977da3e6e8a2dfb
|
|
http://wiki.documentfoundation.org/Clang_plugins
Change-Id: Ieb4fc186490e81ab961c094ca0a7fcdabc0f348f
|
|
Change-Id: I7fdde46334c648ab6cba54210e31fada87dcfb55
|
|
Change-Id: I0fa791733407199db5be2cc9606ac9be1da64188
|
|
Change-Id: I2f98622f152ae0c7ac8d1113d6380f686ac7234c
|
|
Change-Id: I2966fdb5bd98b1ddf718079584acf90a3e3a3700
|
|
But still keep it in case it'd be needed again, or simply
as a starting reference.
Change-Id: If519e1320e1bd6dce7746b57172324b33504051e
|
|
It's mostly there already anyway, no need to duplicate it.
Change-Id: I5b066f90725a064fb0746e1411900e835e3f66c3
|
|
This means that just moving sources to the dir will enable the plugin
action in that source, without modifying any sources, so those changes
would not be accidentally committed when using a rewriter to change
LO sources.
Change-Id: Ic5a9c52dbf1939a1e78ad39ed6691ce3a1f399df
|
|
Now each one registers in its .cxx file.
Change-Id: I811c0d4400c2bdccc1c287269378d7e8ad8743ce
|
|
Change-Id: Ifa1af34121bda9ca3250d09a3b8a46ea226795ed
|
|
Change-Id: I347790e1ca2ee6a27303d3644aa1459ff76ba4b0
|
|
Change-Id: I24bce9dcf46367b5658071ed692d346bb01a83a3
|
|
Change-Id: Ia54040c7ed02e6a2d62dd93890a7d1467dd83a40
|
|
When compiling using icecream, the whole input is one file fed from stdin,
so getBufferName() is "<stdin>", so get the filename from #line directives.
Change-Id: Ie87937b10b0c7e8ddfa57a405b80fd9bde68dd0a
|
|
If the clang binary comes from a package which had been built before
any of our clang related sources were changed the last time, the timestamp
would be older and so there would be no rebuild. So do the stamp handling
the usual way, clang upgrades will work fine, downgrades will not, but
that's the same problem like with downgrading a library and its headers.
To somewhat mitigate the problem (Clang plugin doesn't get cleaned by
'make clean'), include the full Clang version (which includes SVN revision)
in config_clang.h and make all Clang plugin code include that, so
at least configure re-run will trigger a rebuild if necessary.
Change-Id: I993197f79e92e36105092c92c33b2e1db343e975
|
|
Change-Id: I4c8edfc0ee0390d595c43e384bf6e5f595a7b84f
|
|
Change-Id: I9daea65dc28ab13776a7c4319e5d5811515fe160
|
|
Change-Id: I12bf38985ae62756973c05aacf762ae3c405ac9b
|
|
Change-Id: Icb7b017a0c76a6169f0f629bb40bf97449c75837
|
|
Check that only LO's files are processed, as there's no point otherwise.
Also warn about files in workdir/solver/builddir, as those are either
generated or copies. Try to automatically match include files from
solver to srcdir though, as that's where include files are usually
included from :(.
Change-Id: Ie8389e903f623a9d0e75015091acc0da78e76c3a
|
|
|
|
|
|
|
|
Change-Id: Ie4c934b52c4b010703e91f84b83e1fe590b5d187
|
|
...it just appeared to work on machines that happen to have /usr/bin/clang. So
hoist CLANGDIR/CLANGBUILD from compilerplugins/Makefile-clang.mk to configure.ac
instead.
Change-Id: Ie5d9c6bb8e9d0caa1583d78c8693f06b69873095
|