[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