[PATCH] D20681: Add target-specific pre-linking passes to Clang
Anastasia Stulova via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 3 10:06:50 PDT 2016
Anastasia added a comment.
Do you think we could add any test for this change?
================
Comment at: lib/CodeGen/BackendUtil.cpp:657
@@ -627,2 +656,3 @@
+
bool EmitAssemblyHelper::AddEmitPasses(BackendAction Action,
----------------
Remove the empty line please
================
Comment at: lib/CodeGen/CodeGenAction.cpp:169
@@ +168,3 @@
+ std::function<bool(llvm::Module*)>
+ LinkCallBack = [=](llvm::Module *M)->bool {
+ // Link LinkModule into this module if present, preserving its validity.
----------------
Is there any reason for having this as a callback now?
Could we just add a call to prelink passes here above instead without modifying much the original flow?
http://reviews.llvm.org/D20681
More information about the cfe-commits
mailing list