[PATCH] Clarify that the bypassSlowDivision optimization operates on a single BB.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 9 14:42:20 PST 2015


+llvm-commits

Sorry for the noise, still trying to figure out my workflow here.

---------- Forwarded message ----------
From: Justin Lebar <jlebar at google.com>
Date: Wed, Dec 9, 2015 at 2:36 PM
Subject: [PATCH] Clarify that the bypassSlowDivision optimization
operates on a single BB.
To: Hal Finkel <hfinkel at anl.gov>, Tyler Nowicki <tyler.nowicki at gmail.com>


Remove the Function& arg to bypassSlowDivision -- it isn't necessary,
and makes it look like this function operates on a whole function rather
than on one basic block.

Also update some comments to be more explicit.
---
 include/llvm/Transforms/Utils/BypassSlowDivision.h |  8 ++--
 lib/CodeGen/CodeGenPrepare.cpp                     |  2 +-
 lib/Transforms/Utils/BypassSlowDivision.cpp        | 53 ++++++++++------------
 3 files changed, 29 insertions(+), 34 deletions(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Clarify-that-the-bypassSlowDivision-optimization-ope.patch
Type: text/x-patch
Size: 6736 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151209/afdff1f4/attachment.bin>


More information about the llvm-commits mailing list