[PATCH] [MachineSink] Use the real post dominator tree

Jiangning Liu liujiangning1 at gmail.com
Fri Aug 29 03:10:29 PDT 2014


Hi Jingyue,

It sounds not a good idea.

The failure exposed by x86 test case is for performance only rather than correctness, right?

If you have measured x86 performance, and make sure it won't generally affect performance, why can't you add //FIXME in test case instead?

Thanks,
-Jiangning

http://reviews.llvm.org/D4814






More information about the llvm-commits mailing list