Hi Paul,
On 03/06/2013 10:11 PM, Redmond, Paul wrote:
I have updated the patch to not add metadata on loads and stores where the
pointer comes from an alloca. I wonder if the check should be more
conservative and only include pointers coming from Arguments and GlobalValues
(perhaps Constants too?)
I think it's safer that way around for now.
+/// Assuming I is in a parallel loop, return true if I needs
+/// llvm.mem.parallel_loop_access metadata.
+static bool AnnotateParallelLoopAccess(llvm::Instruction *I) {
So maybe this comment should be
"...return true if llvm.mem.parallel_loop_access metadata
can be added safely to I."
--
--Pekka
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
