[llvm-commits] [Review request] test: Add the new feature "loadable_module"
NAKAMURA Takumi
geek4civic at gmail.com
Fri Nov 26 01:26:58 PST 2010
Rafael, good evening.
Thank you to comment!
Two patches attached.
* (updated) 0002-test-Use-SharedLibDir-for-loadable-modules.-On-Cygmi.patch
* (new) 0006-tools-Makefile-Fix-indentation.patch
2010/11/26 Rafael Espíndola <rafael.espindola at gmail.com>:
>> * 0001-test-Add-the-new-feature-loadable_module.patch
>> (application/octet-stream) 2K
>
>
> +# Loadable module
> +# FIXME: This should be supplied by Makefile or autoconf.
> +if sys.platform in ['win32', 'cygwin']:
> + loadable_module = (config.enable_shared == 1)
>
> Why can't you use config.enable_shared in every platform?
I assume other platforms can build loadable modules with ENABLE_PIC=1,
regardless of enable_shared. (and can run one)
4 tests with loadable_module is running validly on ToT.
(eg. linux-elf, freebsd-elf, darwin) (IIRC, --disable-shared)
>> * 0002-test-Use-SharedLibDir-for-loadable-modules.-On-Cygmi.patch
>> (application/octet-stream) 5K
>>
>> It passes $SharedLibDir to lit.
>
> - 'link', 'shlibext', 'ocamlopt', 'llvmdsymutil', 'llvmlibsdir',
> + 'link', 'ocamlopt', 'llvmdsymutil', 'llvmlibsdir',
> + 'llvmshlibdir', 'shlibext',
> 'bugpoint_topts']:
>
> Why the reordering?
It was functionally nonsense, and it has conflicted to Duncan's commit for me.
I regenerated updated patch. I expect someone (or I myself) would tidy it up.
--- a/test/lit.cfg
+++ b/test/lit.cfg
@@ -128,6 +128,7 @@ for line in open(os.path.join(config.llvm_obj_root, 'test',
'site.exp')):
config.substitutions.append(('%llvmgcc_only', site_exp['llvmgcc']))
for sub in ['llvmgcc', 'llvmgxx', 'emitir', 'compile_cxx', 'compile_c',
'link', 'shlibext', 'ocamlopt', 'llvmdsymutil', 'llvmlibsdir',
+ 'llvmshlibdir',
'bugpoint_topts']:
if sub in ('llvmgcc', 'llvmgxx'):
config.substitutions.append(('%' + sub,
>> * 0004-tools-Makefile-Enable-building-lto-edis-and-bugpoint.patch
>> (application/octet-stream) 2K
>>
>> It enables building butpoint-passes, edis and LTO on Cygming.
>> bugpoint-passes can be built with LLVM.dll.
>
> OK, but please reindent the code in the ifdef.
I will fix it up with a separated patch.
Thank you, ...Takumi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-test-Use-SharedLibDir-for-loadable-modules.-On-Cygmi.patch
Type: application/octet-stream
Size: 4072 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101126/f6031c74/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-tools-Makefile-Fix-indentation.patch
Type: application/octet-stream
Size: 1618 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101126/f6031c74/attachment-0001.obj>
More information about the llvm-commits
mailing list