[PATCH] D28196: [X86] Tune bypassing of slow division for Intel CPUs

Nikolai Bozhenov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 07:22:23 PST 2017


n.bozhenov added a comment.

Hi Sanjay,

In https://reviews.llvm.org/D28196#633259, @spatel wrote:

> Can you update the CHECK lines in both of the test files using utils/update_llc_test_checks.py as a preliminary step ahead of this patch (no review required)? This has 3 benefits:
>
> 1. We get tighter checks.
> 2. It's easier to update the test files for this and future transforms.
> 3. It's easier to see the diffs induced by this patch.


Not sure what you mean exactly. Please take a look at https://reviews.llvm.org/D28469. Is it what you mean? Is any additional postprocessing needed after running update_llc_test_checks.py?


https://reviews.llvm.org/D28196





More information about the llvm-commits mailing list