[polly] [Polly] Use separate DT/LI/SE for outlined subfn. NFC. (PR #102460)

via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 9 03:15:50 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 635d20e9e72e25966b196bb012a90e750ae00c6d 67e3230fa4223ec1eb3570871cc8855c834f3d3a --extensions h,cpp -- polly/include/polly/CodeGen/BlockGenerators.h polly/include/polly/CodeGen/IslExprBuilder.h polly/include/polly/CodeGen/IslNodeBuilder.h polly/include/polly/CodeGen/LoopGenerators.h polly/include/polly/CodeGen/LoopGeneratorsGOMP.h polly/include/polly/CodeGen/LoopGeneratorsKMP.h polly/include/polly/Support/ScopHelper.h polly/lib/CodeGen/BlockGenerators.cpp polly/lib/CodeGen/IslExprBuilder.cpp polly/lib/CodeGen/IslNodeBuilder.cpp polly/lib/CodeGen/LoopGeneratorsGOMP.cpp polly/lib/CodeGen/LoopGeneratorsKMP.cpp polly/lib/Support/ScopHelper.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/polly/lib/CodeGen/IslNodeBuilder.cpp b/polly/lib/CodeGen/IslNodeBuilder.cpp
index 8c4f7c759a..3f07f02038 100644
--- a/polly/lib/CodeGen/IslNodeBuilder.cpp
+++ b/polly/lib/CodeGen/IslNodeBuilder.cpp
@@ -597,8 +597,9 @@ void IslNodeBuilder::createForParallel(__isl_take isl_ast_node *For) {
   // TODO: Ideally, we would use the pass manager's TargetLibraryInfoPass and
   // AssumptionAnalysis instead of our own. They contain more target-specific
   // information than we have available here: TargetLibraryInfoImpl can be a
-  // derived class determined by TargetMachine, AssumptionCache can be configured
-  // using a TargetTransformInfo object also derived from TargetMachine.
+  // derived class determined by TargetMachine, AssumptionCache can be
+  // configured using a TargetTransformInfo object also derived from
+  // TargetMachine.
   TargetLibraryInfoImpl BaselineInfoImpl(
       Triple(SubFn->getParent()->getTargetTriple()));
   TargetLibraryInfo CalleeTLI(BaselineInfoImpl, SubFn);
@@ -614,8 +615,6 @@ void IslNodeBuilder::createForParallel(__isl_take isl_ast_node *For) {
   ExprBuilder.switchGeneratedFunc(SubFn, GenDT, GenLI, GenSE);
   Builder.SetInsertPoint(&*LoopBody);
 
-
-  
   // Update the ValueMap to use instructions in the subfunction. Note that
   // "GlobalMap" used in BlockGenerator/IslExprBuilder is a reference to this
   // ValueMap.

``````````

</details>


https://github.com/llvm/llvm-project/pull/102460


More information about the llvm-commits mailing list