[PATCH] Have HasSideEffects() return false for __attribute__((const)) functions

Aaron Ballman aaron.ballman at gmail.com
Mon Apr 6 05:01:46 PDT 2015


LGTM!

~Aaron

On Mon, Apr 6, 2015 at 7:28 AM, Michael Kuperstein
<michael.m.kuperstein at intel.com> wrote:
> Added handling for pure and tweaked the tests a bit.
>
>
> http://reviews.llvm.org/D8548
>
> Files:
>   lib/AST/Expr.cpp
>   test/CodeGen/builtin-assume.c
>   test/Sema/builtin-assume.c
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/



More information about the cfe-commits mailing list