[PATCH] D22189: llvm.noalias - Clang CodeGen - check restrict variable map only for restrict-qualified lvalues

[John McCall via cfe-commits]

rjmccall added inline comments.

================
Comment at: lib/CodeGen/CodeGenFunction.h:2725
@@ -2724,3 +2724,3 @@
   void EmitStoreOfScalar(llvm::Value *Value, Address Addr,
-                         bool Volatile, QualType Ty,
+                         bool Volatile, bool Restrict, QualType Ty,
                          AlignmentSource AlignSource = AlignmentSource::Type,
----------------
majnemer wrote:
> Multiple bool parameters aren't super great. Could we make it an enum and `|` the two properties as needed?  If it's super painful, this could be done later.
Alternatively, is there a reason that the qualifiers on Ty aren't sufficient for this?  If we're passing down the access type anyway, I'm not sure why we need to be separately passing Volatile.


http://reviews.llvm.org/D22189