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 origin/main HEAD --extensions cl,h,cpp --
clang/test/CodeGenOpenCL/builtins-amdgcn.cl
llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp
llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.h
llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp
b/llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp
index f48f36294..a43711b42 100644
--- a/llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp
+++ b/llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp
@@ -3237,8 +3237,9 @@ void GCNScheduleDAGMILive::setTargetOccupancy(unsigned
TargetOccupancy) {
MFI.limitOccupancy(MinOccupancy);
}
-/// Scan for IGLP "mutation-only" instructions (SCHED_GROUP_BARRIER or
IGLP_OPT).
-/// Returns {any in region, strategy ID if such an instruction is IGLP_OPT}.
+/// Scan for IGLP "mutation-only" instructions (SCHED_GROUP_BARRIER or
+/// IGLP_OPT). Returns {any in region, strategy ID if such an instruction is
+/// IGLP_OPT}.
static std::pair<bool, std::optional<AMDGPU::IGLPStrategyID>>
hasIGLPInstrs(ScheduleDAGInstrs *DAG) {
const SIInstrInfo *SII = static_cast<const SIInstrInfo *>(DAG->TII);
@@ -3248,8 +3249,8 @@ hasIGLPInstrs(ScheduleDAGInstrs *DAG) {
if (It == DAG->end())
return {false, std::nullopt};
if (It->getOpcode() == AMDGPU::IGLP_OPT)
- return {true, static_cast<AMDGPU::IGLPStrategyID>(
- It->getOperand(0).getImm())};
+ return {true,
+ static_cast<AMDGPU::IGLPStrategyID>(It->getOperand(0).getImm())};
return {true, std::nullopt};
}
@@ -3275,8 +3276,7 @@ void GCNPostScheduleDAGMILive::schedule() {
// When SCHED_[GROUP_]BARRIER coexists with IGLP_OPT, IGroupLP ignores the
// IGLP_OPT (they are mutually exclusive), so let post-RA scheduling
proceed
// normally.
- if (Strategy == AMDGPU::MFMAValuSpacingOptID &&
- !hasSchedBarrier(this)) {
+ if (Strategy == AMDGPU::MFMAValuSpacingOptID && !hasSchedBarrier(this)) {
HasIGLPInstrs = false;
return;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/190916
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
