search for: enableoldstylefunctions

...* clang.bc.xz 27M with the new bitcode: * clang.bc 70M * clang.bc.xz 26M it looks like a nop run of opt also got a bit faster, from 10.451s to 10.176s. +static void EmitSignedInt64(SmallVectorImpl<uint64_t> &Vals, uint64_t V) { Please start function names with a lower case letter. +EnableOldStyleFunctions("enable-old-style-functions", Maybe something like use-abs-operands is a better name now that this is a global setting, no? > - Jan Cheers, Rafael

On Sun, Sep 30, 2012 at 9:09 AM, Chris Lattner <clattner at apple.com> wrote: > > On Sep 25, 2012, at 5:08 PM, Jan Voung <jvoung at chromium.org> wrote: > > Hi all, > > I've been looking into how to make llvm bitcode files smaller. There is > one simple change that appears to shrink linked bitcode files by about 15%. > See this spreadsheet for some rough

...e to > static bool PushValueAndType(const Value *V, unsigned InstID, > SmallVector<unsigned, 64> &Vals, > ValueEnumerator &VE) { > unsigned ValID = VE.getValueID(V); > - Vals.push_back(ValID); > + if (EnableOldStyleFunctions) > + Vals.push_back(ValID); > + else > + Vals.push_back(InstID - ValID); > if (ValID >= InstID) { > Vals.push_back(VE.getTypeID(V->getType())); > return true; ... > @@ -1164,7 +1191,12 @@ static void WriteInstruction(const Instruction &I, unsigned...

..._t V) { > > Please start function names with a lower case letter. > > Done -- changed this function and most of the "pushValue" functions. I left PushValueAndType alone since that is an existing function and it seemed like that would introduce other unrelated changes. > +EnableOldStyleFunctions("enable-old-style-functions", > > Maybe something like use-abs-operands is a better name now that this > is a global setting, no? > Yes that would be more descriptive. Changed in new patch. > > > - Jan > > Cheers, > Rafael > -------------- next part --...

Hi all, I've been looking into how to make llvm bitcode files smaller. There is one simple change that appears to shrink linked bitcode files by about 15%. See this spreadsheet for some rough data: https://docs.google.com/spreadsheet/ccc?key=0AjRrJHQc4_bddEtJdjdIek5fMDdIdFFIZldZXzdWa0E The change is in how operand ids are encoded in bitcode files. Rather than use an "absolute