[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