Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
H
Hercules
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Wiki
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Snippets
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Package registry
Model registry
Operate
Environments
Terraform modules
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
llvm
Hercules
Merge requests
!207
Loop bound lte
Code
Review changes
Check out branch
Download
Patches
Plain diff
Merged
Loop bound lte
loop-bound-lte
into
main
Overview
0
Commits
5
Pipelines
3
Changes
2
Merged
Xavier Routh
requested to merge
loop-bound-lte
into
main
2 weeks ago
Overview
0
Commits
5
Pipelines
3
Changes
2
Expand
0
0
Merge request reports
Compare
main
version 2
c15146ac
2 weeks ago
version 1
6f161d1e
2 weeks ago
main (base)
and
latest version
latest version
cb5ba51c
5 commits,
2 weeks ago
version 2
c15146ac
3 commits,
2 weeks ago
version 1
6f161d1e
2 commits,
2 weeks ago
2 files
+
7
−
7
Inline
Compare changes
Side-by-side
Inline
Show whitespace changes
Show one file at a time
Files
2
Search (e.g. *.vue) (Ctrl+P)
hercules_opt/src/forkify.rs
+
6
−
5
Options
@@ -106,7 +106,7 @@ pub fn forkify_loop(
else
{
return
false
;
};
// Compute loop variance
let
loop_variance
=
compute_loop_variance
(
editor
,
l
);
let
ivs
=
compute_induction_vars
(
editor
.func
(),
l
,
&
loop_variance
);
@@ -530,10 +530,11 @@ pub fn analyze_phis<'a>(
let
intersection
:
HashSet
<
_
>
=
set1
.intersection
(
&
set2
)
.cloned
()
.collect
();
// If this phi uses any other phis the node is loop dependant,
// we use `phis` because this phi can actually contain the loop iv and its fine.
if
uses_for_dependance
.any
(|
node
|
phis
.contains
(
&
node
)
&&
node
!=
*
phi
)
{
LoopPHI
::
LoopDependant
(
*
phi
)
}
else
if
intersection
.clone
()
.iter
()
.next
()
.is_some
()
{
// // we use `phis` because this phi can actually contain the loop iv and its fine.
// if uses_for_dependance.any(|node| phis.contains(&node) && node != *phi) {
// LoopPHI::LoopDependant(*phi)
// } else
if
intersection
.clone
()
.iter
()
.next
()
.is_some
()
{
// PHIs on the frontier of the uses by the candidate phi, i.e in uses_for_dependance need
// to have headers that postdominate the loop continue latch. The value of the PHI used needs to be defined
// by the time the reduce is triggered (at the end of the loop's internal control).
Loading