[PATCH] D55710: add pragmas to control Software Pipelining optimisation

Alexey Lapshin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 20 05:37:00 PST 2018


alexey.lapshin marked an inline comment as done.
alexey.lapshin added inline comments.


================
Comment at: include/clang/Basic/AttrDocs.td:2673
+
+Check `Software pipelining`, `Modulo scheduling` to get more details on optimisation.
+
----------------
aaron.ballman wrote:
> "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.
I am just unsure which references to include here ... links to wikipedia or some concrete resources do not look good. 

So you are suggesting to spell things out here... Ok, I will prepare short explanation...


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

https://reviews.llvm.org/D55710





More information about the cfe-commits mailing list