[PATCH] D76303: [Pipeliner] Fix the bug in pragma that disables the pipeliner

Alexey Lapshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 18 08:09:02 PDT 2020


avl added inline comments.


================
Comment at: llvm/lib/CodeGen/MachinePipeliner.cpp:295
+    disabledByPragma = false;
+
     if (S->getString() == "llvm.loop.pipeline.initiationinterval") {
----------------
now it becomes depending on order of metadata. It works for "distinct !{!8, !7, !9}" and does not work for distinct !{!8, !9, !7} . This default value should be set in the first line of  setPragmaPipelineOptions (before other instructions):


```
void MachinePipeliner::setPragmaPipelineOptions(MachineLoop &L) {
  disabledByPragma = false;
```


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

https://reviews.llvm.org/D76303





More information about the llvm-commits mailing list