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

Michael Kuperstein michael.m.kuperstein at intel.com
Sun Apr 5 22:19:17 PDT 2015


Thanks, Richard!
Pure functions should not read volatile memory - this is explicitly called out in the gcc docs (" Interesting non-pure functions are functions with infinite loops or those depending on volatile memory"). Since both you and Hal seem certain it's fine, I'll update the patch and add that.


http://reviews.llvm.org/D8548

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list