[PATCH] Fix regression introduced in r205566.
Rui Ueyama
ruiu at google.com
Tue May 13 12:20:53 PDT 2014
================
Comment at: lib/Core/Resolver.cpp:66-68
@@ -69,1 +65,5 @@
+ llvm::dbgs() << "undefAdded: " << undefAdded << "\n";
+ if (undefAdded) {
+ _context.getInputGraph().notifyProgress();
+ }
}
----------------
Shankar Kalpathi Easwaran wrote:
> it looks like notifyProgress is called only if there is a undefined atom added, can we do it in doUndefinedAtom ? This way there would be no need to change much.
I don't want to call notifyProgress in doUndefinedAtom. If we do, notifyProgress() would be called for each undefined atom. That's not wrong but would be slow. I want call it only once.
http://reviews.llvm.org/D3734
More information about the llvm-commits
mailing list