[llvm-commits] -fno-rtti for vmcore

[Török Edwin]

Hi,

vmcore doesn't need rtti either, I think it was an overly conservative
flag setting in the makefile at fault,
where REQUIRES_EH automatically turned on REQUIRES_RTTI (probably
because Makefiles didn't
use to properly specify REQUIRES_RTTI):
   # If the library requires EH, it also requires RTTI.
  REQUIRES_RTTI := 1

But my gcc manual says that "Note that exception handling uses the same
information, but it will
generate it as needed.". And in fact if I remove the REQUIRES_EH ->
REQUIRES_RTTI dependency,
and REQUIRES_RTTI from VMCore and unittests, bugpoint and everything
else still builds.
make check runs fine too.

So is the attached patch safe to apply?

P.S.: support and system still need rtti of course due to tblgen.

Best regards,
--Edwin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nortti.patch
Type: text/x-diff
Size: 1873 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20100125/e0804d25/attachment.patch>