aaron.ballman added inline comments.

================
Comment at: include/clang/Basic/AttrDocs.td:2673
+
+Check `Software pipelining`, `Modulo scheduling` to get more details on 
optimisation.
+
----------------
"Check" -- check where? Perhaps "Search for" but I still don't think that's 
very satisfying for our documentation as there's no telling what users will 
find or how it relates (if at all) to this feature.

on optimisation -> on this optimization

I find this sentence doesn't really direct me to much of value (research 
presentations and class lectures, mostly), so I'd rather see the important 
details spelled out in our docs so that users don't have to guess as much.


================
Comment at: include/clang/Basic/AttrDocs.td:2677-2678
+`language extensions
+<http://clang.llvm.org/docs/LanguageExtensions.html#extensions-for-loop-hint-optimizations>`_
+for further details, including limitations of the pipeline hints.
+  }];
----------------
There is nothing in the linked documentation that describes pipeline initiation 
intervals, so I'm still left wondering how to pick a value for the attribute 
argument. I think the docs need a bit more detail for what that value means and 
how it impacts the user's code.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55710/new/

https://reviews.llvm.org/D55710



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to