diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2014-08-21 22:31:48 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2014-08-21 22:31:48 +0000 |
| commit | 2cdea4c41e875b2e8906bdd3eb9c2886a9f7c69a (patch) | |
| tree | 3a8be4ee7dc51f8c57c462befaa945326ce3130b /clang/lib/CodeGen/CodeGenFunction.h | |
| parent | 6674b095b85df6cc908029e7ed5c2e71f69e3c30 (diff) | |
| download | llvm-2cdea4c41e875b2e8906bdd3eb9c2886a9f7c69a.zip llvm-2cdea4c41e875b2e8906bdd3eb9c2886a9f7c69a.tar.gz llvm-2cdea4c41e875b2e8906bdd3eb9c2886a9f7c69a.tar.bz2 | |
name change: isPow2DivCheap -> isPow2SDivCheap
isPow2DivCheap
That name doesn't specify signed or unsigned.
Lazy as I am, I eventually read the function and variable comments. It turns out that this is strictly about signed div. But I discovered that the comments are wrong:
srl/add/sra
is not the general sequence for signed integer division by power-of-2. We need one more 'sra':
sra/srl/add/sra
That's the sequence produced in DAGCombiner. The first 'sra' may be removed when dividing by exactly '2', but that's a special case.
This patch corrects the comments, changes the name of the flag bit, and changes the name of the accessor methods.
No functional change intended.
Differential Revision: http://reviews.llvm.org/D5010
llvm-svn: 216237
Diffstat (limited to 'clang/lib/CodeGen/CodeGenFunction.h')
0 files changed, 0 insertions, 0 deletions