[TSAR, Parallel, DVMH] Fix, allow IPO if there are no parents of IPO root.

kaniandr · kaniandr · commit 077dc67bc905 · 2021-07-29T21:43:32.000+03:00
diff --git a/lib/Transform/Clang/DVMHSMAutoPar.cpp b/lib/Transform/Clang/DVMHSMAutoPar.cpp
@@ -2503,8 +2503,11 @@ bool ClangDVMHSMParallelization::runOnModule(llvm::Module &M) {
       return;
     }
     // Enable IPO only if all children of the IPO root is valid.
-    if (PI == &mIPORoot && !PD->children().empty() && !PD->isRequired()) {
-      mIPORoot.invalidate();
+    if (PI == &mIPORoot) {
+      if (!PD->children().empty())
+        mIPORoot.invalidate();
+      else
+        mIPORoot.skip();
       return;
     }
     auto PIItr{DeferredPragmas.find_as(PI)};

Original file line number	Diff line number	Diff line change
`@@ -2503,8 +2503,11 @@ bool ClangDVMHSMParallelization::runOnModule(llvm::Module &M) {`
`2503`	`2503`	`return;`
`2504`	`2504`	`}`
`2505`	`2505`	`// Enable IPO only if all children of the IPO root is valid.`
`2506`		`- if (PI == &mIPORoot && !PD->children().empty() && !PD->isRequired()) {`
`2507`		`- mIPORoot.invalidate();`
	`2506`	`+ if (PI == &mIPORoot) {`
	`2507`	`+ if (!PD->children().empty())`
	`2508`	`+ mIPORoot.invalidate();`
	`2509`	`+ else`
	`2510`	`+ mIPORoot.skip();`
`2508`	`2511`	`return;`
`2509`	`2512`	`}`
`2510`	`2513`	`auto PIItr{DeferredPragmas.find_as(PI)};`