[PATCH] D35315: [Dominators] Make IsPostDominator a template parameter

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 16 08:18:37 PDT 2017


kuhar added a comment.

I cannot reproduce this failure on my machine. Could someone be able to point me to a build script that makes x86_64 hit the problem?

I would guess that the problem has something to do with explicit template instantiation in MachineDominators.cpp and DominanaceFrontier.cpp. The only difference between these two files and Dominators.cpp, which was edited the same way, is that explicit instantiations are put inside the llvm namespace.
Here's a patch that puts them in global scope: https://reviews.llvm.org/D35461

Please do let me know if applying the patch fixes the failure.


Repository:
  rL LLVM

https://reviews.llvm.org/D35315





More information about the llvm-commits mailing list