[llvm-commits] Fwd: [PATCHES] fix duplication and FIXMEs in JIT code
Jim Grosbach
grosbach at apple.com
Fri May 6 12:00:22 PDT 2011
Looks reasonable to me. Thanks!
-Jim
On Apr 24, 2011, at 3:20 PM, nobled wrote:
> Whoops, forgot to join the mailing list, so this got stuck in the
> moderation queue...
>
> ---------- Forwarded message ----------
> From: nobled <nobled at dreamwidth.org>
> Date: Sun, Apr 24, 2011 at 11:05 AM
> Subject: [PATCHES] fix duplication and FIXMEs in JIT code
> To: llvm-commits at cs.uiuc.edu
>
>
> (Just so you know, I do have commit access.)
>
> These patches address this comment from the MCJIT code:
>
> // FIXME: This should be lifted out of here, it isn't something which should
> // be part of the JIT policy, rather the burden for this selection should be
> // pushed to clients.
> ====
> [PATCH 1/3] ExecutionEngine: fix JIT/MCJIT selectTarget() duplication
>
> (Merges the near-identical files lib/EE/JIT/TargetSelect.cpp and
> lib/EE/MCJIT/TargetSelect.cpp into lib/EE/TargetSelect.cpp.)
>
> This prepares for making JITCtor/MCJITCtor take a
> TargetMachine* directly from clients like EngineBuilder.
> ====
> [PATCH 2/3] ExecutionEngine: push TargetMachine creation into clients
>
> In particular, into EngineBuilder. This should only impact
> the private API between the EE and EB classes, not external
> clients, since JITCtor and MCJITCtor are both protected members.
> ====
> [PATCH 3/3] ExecutionEngine: move createJIT definition
>
> As an ExecutionEngine class function, its definition
> really belongs in ExecutionEngine.cpp instead of in JIT.cpp.
> ====
>
> As for this comment:
>
> // Try to register the program as a source of symbols to resolve against.
> //
> // FIXME: Don't do this here.
> sys::DynamicLibrary::LoadLibraryPermanently(0, NULL);
>
> where in the stack *should* that go, ideally?
> <0001-ExecutionEngine-fix-JIT-MCJIT-selectTarget-duplicati.patch.txt><0002-ExecutionEngine-push-TargetMachine-creation-into-cli.patch.txt><0003-ExecutionEngine-move-createJIT-definition.patch.txt>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list