[PATCH] D103751: [Polly][Isl] Removing nullptr constructor from C++ bindings. NFC.

[Riccardo Mori via Phabricator]

patacca created this revision.
Herald added a reviewer: bollu.
Herald added subscribers: ormris, asbirlea, steven_wu, hiraditya.
patacca added a reviewer: Meinersbur.
patacca added a project: Polly.
patacca added a subscriber: pollydev.
patacca published this revision for review.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

`isl-noexceptions.h` has been generated by this https://github.com/patacca/isl/commit/a7e00bea38f251a4bcf5c2c6ce5fa7ee5f661528

Changes made:

- Removed `std::nullptr_t` constructor from all the classes in the isl C++ bindings.


Repository:
rG LLVM Github Monorepo

https://reviews.llvm.org/D103751

Files:
polly/include/polly/ScopInfo.h
polly/lib/Analysis/ScopBuilder.cpp
polly/lib/Analysis/ScopInfo.cpp
polly/lib/CodeGen/IslAst.cpp
polly/lib/External/isl/include/isl/isl-noexceptions.h
polly/lib/Support/ISLTools.cpp
polly/lib/Transform/DeLICM.cpp
polly/lib/Transform/FlattenAlgo.cpp
polly/lib/Transform/FlattenSchedule.cpp
polly/lib/Transform/ForwardOpTree.cpp
polly/lib/Transform/ScheduleOptimizer.cpp
polly/lib/Transform/ZoneAlgo.cpp
polly/unittests/DeLICM/DeLICMTest.cpp

[Riccardo Mori via Phabricator]

patacca updated this revision to Diff 350114.
patacca added a comment.

Rebase to main

Repository:
rG LLVM Github Monorepo

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

[Michael Kruse via Phabricator]

Meinersbur accepted this revision.
Meinersbur added a comment.
This revision is now accepted and ready to land.

LGTM.

Could you mention in the commit message/summary that the goal/motivation is to make the isl-noexceptions.h converge towards the official bindings?



================
Comment at: polly/lib/Transform/ScheduleOptimizer.cpp:1702

- ~IslScheduleOptimizerWrapperPass() override { releaseMemory(); }
-
----------------
This might have been there to ensure the order in which the objects are freed. `LastSchedule` must be free'd before `IslCtx`.

However, destruction order is defined to be in reverse of the field declaration order, so indeed what we want.

[Riccardo Mori via Phabricator]

patacca edited the summary of this revision.

[Riccardo Mori via Phabricator]

This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGbe5e2fc7bf78: [Polly][Isl] Removing nullptr constructor from C++ bindings. NFC. (authored by patacca).

Repository:
rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103751

[Riccardo Mori via Phabricator]

patacca added a reverting change: rGf60ea691a94b: Revert "[Polly][Isl] Removing nullptr constructor from C++ bindings. NFC.".

[Riccardo Mori via Phabricator]

patacca reopened this revision.

This revision is now accepted and ready to land.

[Riccardo Mori via Phabricator]

patacca updated this revision to Diff 350710.
patacca added a comment.
Herald added subscribers: kbarton, nemanjai.

Replacing calls to nullptr constructor of isl::schedule to the default constructor

Repository:
rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103751

Files:
polly/lib/CodeGen/PPCGCodeGeneration.cpp


Index: polly/lib/CodeGen/PPCGCodeGeneration.cpp
===================================================================
--- polly/lib/CodeGen/PPCGCodeGeneration.cpp
+++ polly/lib/CodeGen/PPCGCodeGeneration.cpp
@@ -178,7 +178,7 @@
/// [params] -> { Stmt_phantom[] -> scalar_to_kill[] }
isl::union_map MustKills;

- MustKillsInfo() : KillsSchedule(nullptr) {}
+ MustKillsInfo() : KillsSchedule() {}
};

/// Check if SAI's uses are entirely contained within Scop S.
@@ -227,7 +227,7 @@
// - filter: "[control] -> { }"
// So, we choose to not create this to keep the output a little nicer,
// at the cost of some code complexity.
- Info.KillsSchedule = nullptr;
+ Info.KillsSchedule = {};

for (isl::id &ToKillId : KillMemIds) {
isl::id KillStmtId = isl::id::alloc(

[Riccardo Mori via Phabricator]

patacca updated this revision to Diff 350711.

patacca added a comment.

Rebase to main

Repository:
rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103751

Files:

polly/include/polly/ScopInfo.h
polly/lib/Analysis/ScopBuilder.cpp
polly/lib/Analysis/ScopInfo.cpp
polly/lib/CodeGen/IslAst.cpp
polly/lib/CodeGen/PPCGCodeGeneration.cpp

[Riccardo Mori via Phabricator]

This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.

Closed by commit rG9b41d0958e78: [Polly][Isl] Removing nullptr constructor from C++ bindings. NFC. (authored by patacca).