[llvm-commits] [llvm] r55711 - /llvm/trunk/lib/Transforms/IPO/InlineSimple.cpp
Bill Wendling
isanbard at gmail.com
Wed Sep 3 12:54:47 PDT 2008
On Wed, Sep 3, 2008 at 11:46 AM, Devang Patel <dpatel at apple.com> wrote:
> --- llvm/trunk/lib/Transforms/IPO/InlineSimple.cpp (original)
> +++ llvm/trunk/lib/Transforms/IPO/InlineSimple.cpp Wed Sep 3 13:46:35 2008
> @@ -65,7 +65,7 @@
>
> for (Module::iterator I = M.begin(), E = M.end();
> I != E; ++I)
> - if (!I->isDeclaration() && I->getNotes() == FN_NOTE_NoInline)
> + if (!I->isDeclaration() && I->getNotes() & FN_NOTE_NoInline)
I know that this is technically correct, but could you put parentheses
around the bitwise "and"? It makes it more readable if you don't have
the C operator precedence table memorized. :-)
-bw
> NeverInline.insert(I);
>
More information about the llvm-commits
mailing list