[PATCH] llvm.noalias - CaptureTracking needs to look through them

Philip Reames listmail at philipreames.com
Tue May 12 15:49:40 PDT 2015


LGTM w.r.t. code.

I would prefer the intrinsic definition be marked nocapture if it's legal to do so.  There's too much chance of someone checking only an attribute somewhere.  Better to be complete as we can.

Also, I don't believe the parameter on stripPointerCasts is still needed per discussion in another thread right?


http://reviews.llvm.org/D9386

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






More information about the llvm-commits mailing list