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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
