[PATCH] D17570: Introduce DominanceFrontierAnalysis to the new PassManager to compute DominanceFrontier. NFC

Hongbin Zheng via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 05:30:28 PST 2016


etherzhhb added a comment.

In http://reviews.llvm.org/D17570#361584, @grosser wrote:

> Hi ether,
>
> this patch looks good to me as well. I know we currently have no actual dominance frontier tests, but I think this would be a good occasion to add one. Domfrontier is hopefully removed at some point in time, but until this time comes it is good to have at least some basic test coverage.
>
> Best,
> Tobias


I am going to add one from RegionInfo tests


Repository:
  rL LLVM

http://reviews.llvm.org/D17570





More information about the llvm-commits mailing list