[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