[PATCH] D42717: [JumpThreading] sync DT for LVI analysis (PR 36133)

Brian Rzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 2 12:57:30 PST 2018


brzycki added a comment.

In https://reviews.llvm.org/D42717#996670, @kuhar wrote:

> That seems to be within noise and not an issue


I tend to agree. Just for my own curiosity I also ran a check to see if my proposed patch for https://reviews.llvm.org/D34135 changed compile time:

  "compile_time": 1268.6039999999998, # tip patched with proposed fix for both this issue and D34135

It's also still within noise levels I think.


Repository:
  rL LLVM

https://reviews.llvm.org/D42717





More information about the llvm-commits mailing list