Skip to content

Empty test (just a test...) #65439

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from
Closed

Empty test (just a test...) #65439

wants to merge 2 commits into from

Conversation

joker-eph
Copy link
Collaborator

@joker-eph joker-eph commented Sep 6, 2023

Test description...

Line 2

@joker-eph joker-eph requested review from a team as code owners September 6, 2023 06:24
@github-actions github-actions bot added the mlir label Sep 6, 2023
@joker-eph joker-eph force-pushed the empty-test branch 2 times, most recently from 46a0303 to 22d035b Compare September 6, 2023 06:29
@joker-eph joker-eph closed this Sep 6, 2023
@joker-eph
Copy link
Collaborator Author

@llvm/pr-subscribers-mlir-async

Test description...

Line 2

Patch is 4.74 MiB, truncated to 10kB below, full version: https://github.com//pull/65439.diff

Changes
diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index a453601d7ad93d1..3d25ce9488f126e 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -46,21 +46,33 @@ quality.
 Stay Informed
 -------------

-Developers should stay informed by reading the LLVM Discourse forums.
-If you are doing anything more than just casual work on LLVM, it is suggested that you also
-subscribe to the "commits" mailing list for the subproject you're interested in,
+Developers should stay informed by reading the LLVM Discourse forums
and subscribing
+to the categories of interest for notifications.
+
+Paying attention to changes being made by others is a good way to see what other people
+are interested in and watching the flow of the project as a whole.
+
+Contibutions to the project are made through :ref:GitHub Pull Requests <github-reviews>.
+You can subscribe to notification for areas of the codebase by joining
+one of the pr-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+GitHub teams. This mapping <https://github.com/llvm/llvm-project/blob/main/.github/new-prs-labeler.yml>_
+indicates which team is associated with a particular paths in the repository.
+
+You can also subscribe to the "commits" mailing list for a subproject you're interested in,
such as llvm-commits <http://lists.llvm.org/mailman/listinfo/llvm-commits>, cfe-commits <http://lists.llvm.org/mailman/listinfo/cfe-commits>, or lldb-commits -<http://lists.llvm.org/mailman/listinfo/lldb-commits>_. Reading the
-"commits" list and paying attention to changes being made by others is a good
-way to see what other people are interested in and watching the flow of the
-project as a whole.

-We recommend that active developers monitor incoming issues to our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>_ and preferably subscribe to the llvm-bugs +<http://lists.llvm.org/mailman/listinfo/lldb-commits>.
+
+Missing features and bugs are tracked through our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>

+and assigned labels. We recommend that active developers monitor incoming issues.
+You can subscribe for notification for specific components by joining
+one of the issue-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+teams.
+You may also subscribe to the llvm-bugs <http://lists.llvm.org/mailman/listinfo/llvm-bugs>_ email list to keep track
-of bugs and enhancements occurring in LLVM. We really appreciate people who are
-proactive at catching incoming bugs in their components and dealing with them
+of bugs and enhancements occurring in the entire project. We really appreciate people
+who are proactive at catching incoming bugs in their components and dealing with them
promptly.

Please be aware that all public LLVM mailing lists and discourse forums are public and archived, and
diff --git a/llvm/lib/Analysis/AliasAnalysis.cpp b/llvm/lib/Analysis/AliasAnalysis.cpp
index 75fe68a80ab6085..579087857b7eb0e 100644
--- a/llvm/lib/Analysis/AliasAnalysis.cpp
+++ b/llvm/lib/Analysis/AliasAnalysis.cpp
@@ -56,8 +56,8 @@

using namespace llvm;

-STATISTIC(NumNoAlias, "Number of NoAlias results");
-STATISTIC(NumMayAlias, "Number of MayAlias results");
+STATISTIC(NumNoAlias, "Number of NoAlias results");
+STATISTIC(NumMayAlias, "Number of MayAlias results");
STATISTIC(NumMustAlias, "Number of MustAlias results");

namespace llvm {
@@ -116,8 +116,8 @@ AliasResult AAResults::alias(const MemoryLocation &LocA,
if (EnableAATrace) {
for (unsigned I = 0; I < AAQI.Depth; ++I)
dbgs() << " ";

  • dbgs() << "Start " << *LocA.Ptr << " @ " << LocA.Size << ", "
  •       << *LocB.Ptr << " @ " << LocB.Size << "\n";
    
  • dbgs() << "Start " << *LocA.Ptr << " @ " << LocA.Size << ", " << *LocB.Ptr

  •       << " @ " << LocB.Size << "\n";
    

    }

    AAQI.Depth++;
    @@ -131,8 +131,8 @@ AliasResult AAResults::alias(const MemoryLocation &LocA,
    if (EnableAATrace) {
    for (unsigned I = 0; I < AAQI.Depth; ++I)
    dbgs() << " ";

  • dbgs() << "End " << *LocA.Ptr << " @ " << LocA.Size << ", "
  •       << *LocB.Ptr << " @ " << LocB.Size << " = " << Result << "\n";
    
  • dbgs() << "End " << *LocA.Ptr << " @ " << LocA.Size << ", " << *LocB.Ptr

  •       << " @ " << LocB.Size << " = " << Result << "\n";
    

    }

    if (AAQI.Depth == 0) {
    @@ -584,7 +584,8 @@ ModRefInfo AAResults::getModRefInfo(const AtomicCmpXchgInst *CX,
    ModRefInfo AAResults::getModRefInfo(const AtomicRMWInst *RMW,
    const MemoryLocation &Loc,
    AAQueryInfo &AAQI) {

  • // Acquire/Release atomicrmw has properties that matter for arbitrary addresses.
  • // Acquire/Release atomicrmw has properties that matter for arbitrary
  • // addresses.
    if (isStrongerThanMonotonic(RMW->getOrdering()))
    return ModRefInfo::ModRef;

@@ -646,8 +647,7 @@ ModRefInfo AAResults::getModRefInfo(const Instruction *I,
/// with a smarter AA in place, this test is just wasting compile time.
ModRefInfo AAResults::callCapturesBefore(const Instruction *I,
const MemoryLocation &MemLoc,

  •                                     DominatorTree *DT,
    
  •                                     AAQueryInfo &AAQI) {
    
  •                                     DominatorTree *DT, AAQueryInfo &AAQI) {
    
    if (!DT)
    return ModRefInfo::ModRef;

@@ -718,7 +718,7 @@ bool AAResults::canInstructionRangeModRef(const Instruction &I1,
"Instructions not in same basic block!");
BasicBlock::const_iterator I = I1.getIterator();
BasicBlock::const_iterator E = I2.getIterator();

  • ++E; // Convert from inclusive to exclusive range.
  • ++E; // Convert from inclusive to exclusive range.

    for (; I != E; ++I) // Check every instruction in range
    if (isModOrRefSet(getModRefInfo(&*I, Loc) & Mode))
    diff --git a/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp b/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    index a551ea6b69c5a40..9811af81642db4f 100644
    --- a/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    +++ b/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    @@ -24,7 +24,8 @@ static cl::opt PrintAll("print-all-alias-modref-info", cl::ReallyHidden);

static cl::opt PrintNoAlias("print-no-aliases", cl::ReallyHidden);
static cl::opt PrintMayAlias("print-may-aliases", cl::ReallyHidden);
-static cl::opt PrintPartialAlias("print-partial-aliases", cl::ReallyHidden);
+static cl::opt PrintPartialAlias("print-partial-aliases",

  •                                   cl::ReallyHidden);
    

static cl::opt PrintMustAlias("print-must-aliases", cl::ReallyHidden);

static cl::opt PrintNoModRef("print-no-modref", cl::ReallyHidden);
@@ -68,9 +69,9 @@ static void PrintResults(AliasResult AR, bool P,
}
}

-static inline void PrintModRefResults(

  • const char *Msg, bool P, Instruction *I,
  • std::pair<const Value *, Type *> Loc, Module *M) {
    +static inline void PrintModRefResults(const char *Msg, bool P, Instruction *I,
  •                                  std::pair<const Value *, Type *> Loc,
    
  •                                  Module *M) {
    
    if (PrintAll || P) {
    errs() << " " << Msg << ": Ptr: ";
    Loc.second->print(errs(), false, /* NoDetails */ true);
    @@ -115,8 +116,8 @@ void AAEvaluator::runInternal(Function &F, AAResults &AA) {
    Pointers.insert({LI->getPointerOperand(), LI->getType()});
    Loads.insert(LI);
    } else if (auto *SI = dyn_cast(&Inst)) {
  •  Pointers.insert({SI->getPointerOperand(),
    
  •                   SI->getValueOperand()->getType()});
    
  •  Pointers.insert(
    
  •      {SI->getPointerOperand(), SI->getValueOperand()->getType()});
     Stores.insert(SI);
    
    } else if (auto *CB = dyn_cast(&Inst))
    Calls.insert(CB);
    @@ -171,7 +172,8 @@ void AAEvaluator::runInternal(Function &F, AAResults &AA) {
    ++MayAliasCount;
    break;
    case AliasResult::PartialAlias:
  •      PrintLoadStoreResults(AR, PrintPartialAlias, Load, Store, F.getParent());
    
  •      PrintLoadStoreResults(AR, PrintPartialAlias, Load, Store,
    
  •                            F.getParent());
         ++PartialAliasCount;
         break;
       case AliasResult::MustAlias:
    

@@ -349,7 +351,7 @@ class AAEvalLegacyPass : public FunctionPass {
return false;
}
};
-}
+} // namespace llvm

char AAEvalLegacyPass::ID = 0;
INITIALIZE_PASS_BEGIN(AAEvalLegacyPass, "aa-eval",
diff --git a/llvm/lib/Analysis/AliasSetTracker.cpp b/llvm/lib/Analysis/AliasSetTracker.cpp
index 91b889116dfa2d3..8c65c001d2fe311 100644
--- a/llvm/lib/Analysis/AliasSetTracker.cpp
+++ b/llvm/lib/Analysis/AliasSetTracker.cpp
@@ -49,7 +49,7 @@ void AliasSet::mergeSetIn(AliasSet &AS, AliasSetTracker &AST,
bool WasMustAlias = (Alias == SetMustAlias);
// Update the alias and access types of this set...
Access |= AS.Access;

  • Alias |= AS.Alias;
  • Alias |= AS.Alias;

    if (Alias == SetMustAlias) {
    // Check that these two merged sets really are must aliases. Since both
    @@ -73,7 +73,7 @@ void AliasSet::mergeSetIn(AliasSet &AS, AliasSetTracker &AST,
    }

    bool ASHadUnknownInsts = !AS.UnknownInsts.empty();

  • if (UnknownInsts.empty()) { // Merge call sites...
  • if (UnknownInsts.empty()) { // Merge call sites...
    if (ASHadUnknownInsts) {
    std::swap(UnknownInsts, AS.UnknownInsts);
    addRef();
    @@ -107,8 +107,8 @@ void AliasSetTracker::removeAliasSet(AliasSet *AS) {
    Fwd->dropRef(*this);
    AS->Forward = nullptr;
    } else // Update TotalMayAliasSetSize only if not forwarding.
  •  if (AS->Alias == AliasSet::SetMayAlias)
    
  •    TotalMayAliasSetSize -= AS->size();
    
  • if (AS->Alias == AliasSet::SetMayAlias)

  •  TotalMayAliasSetSize -= AS->size();
    

    AliasSets.erase(AS);
    // If we've removed the saturated alias set, set saturated ma

@joker-eph
Copy link
Collaborator Author

@llvm/pr-subscribers-mlir-async

Test description...

Line 2

Patch is 4.74 MiB, truncated to 10kB below, full version: https://github.com/llvm/llvm-project/pull/65439.diff

Changes
diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index a453601d7ad93d1..3d25ce9488f126e 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -46,21 +46,33 @@ quality.
 Stay Informed
 -------------

-Developers should stay informed by reading the LLVM Discourse forums.
-If you are doing anything more than just casual work on LLVM, it is suggested that you also
-subscribe to the "commits" mailing list for the subproject you're interested in,
+Developers should stay informed by reading the LLVM Discourse forums
and subscribing
+to the categories of interest for notifications.
+
+Paying attention to changes being made by others is a good way to see what other people
+are interested in and watching the flow of the project as a whole.
+
+Contibutions to the project are made through :ref:GitHub Pull Requests <github-reviews>.
+You can subscribe to notification for areas of the codebase by joining
+one of the pr-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+GitHub teams. This mapping <https://github.com/llvm/llvm-project/blob/main/.github/new-prs-labeler.yml>_
+indicates which team is associated with a particular paths in the repository.
+
+You can also subscribe to the "commits" mailing list for a subproject you're interested in,
such as llvm-commits <http://lists.llvm.org/mailman/listinfo/llvm-commits>, cfe-commits <http://lists.llvm.org/mailman/listinfo/cfe-commits>, or lldb-commits -<http://lists.llvm.org/mailman/listinfo/lldb-commits>_. Reading the
-"commits" list and paying attention to changes being made by others is a good
-way to see what other people are interested in and watching the flow of the
-project as a whole.

-We recommend that active developers monitor incoming issues to our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>_ and preferably subscribe to the llvm-bugs +<http://lists.llvm.org/mailman/listinfo/lldb-commits>.
+
+Missing features and bugs are tracked through our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>

+and assigned labels. We recommend that active developers monitor incoming issues.
+You can subscribe for notification for specific components by joining
+one of the issue-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+teams.
+You may also subscribe to the llvm-bugs <http://lists.llvm.org/mailman/listinfo/llvm-bugs>_ email list to keep track
-of bugs and enhancements occurring in LLVM. We really appreciate people who are
-proactive at catching incoming bugs in their components and dealing with them
+of bugs and enhancements occurring in the entire project. We really appreciate people
+who are proactive at catching incoming bugs in their components and dealing with them
promptly.

Please be aware that all public LLVM mailing lists and discourse forums are public and archived, and
diff --git a/llvm/lib/Analysis/AliasAnalysis.cpp b/llvm/lib/Analysis/AliasAnalysis.cpp
index 75fe68a80ab6085..579087857b7eb0e 100644
--- a/llvm/lib/Analysis/AliasAnalysis.cpp
+++ b/llvm/lib/Analysis/AliasAnalysis.cpp
@@ -56,8 +56,8 @@

using namespace llvm;

-STATISTIC(NumNoAlias, "Number of NoAlias results");
-STATISTIC(NumMayAlias, "Number of MayAlias results");
+STATISTIC(NumNoAlias, "Number of NoAlias results");
+STATISTIC(NumMayAlias, "Number of MayAlias results");
STATISTIC(NumMustAlias, "Number of MustAlias results");

namespace llvm {
@@ -116,8 +116,8 @@ AliasResult AAResults::alias(const MemoryLocation &LocA,
if (EnableAATrace) {
for (unsigned I = 0; I < AAQI.Depth; ++I)
dbgs() << " ";

  • dbgs() << "Start " << *LocA.Ptr << " @ " << LocA.Size << ", "
  •       << *LocB.Ptr << " @ " << LocB.Size << "\n";
    
  • dbgs() << "Start " << *LocA.Ptr << " @ " << LocA.Size << ", " << *LocB.Ptr

  •       << " @ " << LocB.Size << "\n";
    

    }

    AAQI.Depth++;
    @@ -131,8 +131,8 @@ AliasResult AAResults::alias(const MemoryLocation &LocA,
    if (EnableAATrace) {
    for (unsigned I = 0; I < AAQI.Depth; ++I)
    dbgs() << " ";

  • dbgs() << "End " << *LocA.Ptr << " @ " << LocA.Size << ", "
  •       << *LocB.Ptr << " @ " << LocB.Size << " = " << Result << "\n";
    
  • dbgs() << "End " << *LocA.Ptr << " @ " << LocA.Size << ", " << *LocB.Ptr

  •       << " @ " << LocB.Size << " = " << Result << "\n";
    

    }

    if (AAQI.Depth == 0) {
    @@ -584,7 +584,8 @@ ModRefInfo AAResults::getModRefInfo(const AtomicCmpXchgInst *CX,
    ModRefInfo AAResults::getModRefInfo(const AtomicRMWInst *RMW,
    const MemoryLocation &Loc,
    AAQueryInfo &AAQI) {

  • // Acquire/Release atomicrmw has properties that matter for arbitrary addresses.
  • // Acquire/Release atomicrmw has properties that matter for arbitrary
  • // addresses.
    if (isStrongerThanMonotonic(RMW->getOrdering()))
    return ModRefInfo::ModRef;

@@ -646,8 +647,7 @@ ModRefInfo AAResults::getModRefInfo(const Instruction *I,
/// with a smarter AA in place, this test is just wasting compile time.
ModRefInfo AAResults::callCapturesBefore(const Instruction *I,
const MemoryLocation &MemLoc,

  •                                     DominatorTree *DT,
    
  •                                     AAQueryInfo &AAQI) {
    
  •                                     DominatorTree *DT, AAQueryInfo &AAQI) {
    
    if (!DT)
    return ModRefInfo::ModRef;

@@ -718,7 +718,7 @@ bool AAResults::canInstructionRangeModRef(const Instruction &I1,
"Instructions not in same basic block!");
BasicBlock::const_iterator I = I1.getIterator();
BasicBlock::const_iterator E = I2.getIterator();

  • ++E; // Convert from inclusive to exclusive range.
  • ++E; // Convert from inclusive to exclusive range.

    for (; I != E; ++I) // Check every instruction in range
    if (isModOrRefSet(getModRefInfo(&*I, Loc) & Mode))
    diff --git a/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp b/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    index a551ea6b69c5a40..9811af81642db4f 100644
    --- a/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    +++ b/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    @@ -24,7 +24,8 @@ static cl::opt PrintAll("print-all-alias-modref-info", cl::ReallyHidden);

static cl::opt PrintNoAlias("print-no-aliases", cl::ReallyHidden);
static cl::opt PrintMayAlias("print-may-aliases", cl::ReallyHidden);
-static cl::opt PrintPartialAlias("print-partial-aliases", cl::ReallyHidden);
+static cl::opt PrintPartialAlias("print-partial-aliases",

  •                                   cl::ReallyHidden);
    

static cl::opt PrintMustAlias("print-must-aliases", cl::ReallyHidden);

static cl::opt PrintNoModRef("print-no-modref", cl::ReallyHidden);
@@ -68,9 +69,9 @@ static void PrintResults(AliasResult AR, bool P,
}
}

-static inline void PrintModRefResults(

  • const char *Msg, bool P, Instruction *I,
  • std::pair<const Value *, Type *> Loc, Module *M) {
    +static inline void PrintModRefResults(const char *Msg, bool P, Instruction *I,
  •                                  std::pair<const Value *, Type *> Loc,
    
  •                                  Module *M) {
    
    if (PrintAll || P) {
    errs() << " " << Msg << ": Ptr: ";
    Loc.second->print(errs(), false, /* NoDetails */ true);
    @@ -115,8 +116,8 @@ void AAEvaluator::runInternal(Function &F, AAResults &AA) {
    Pointers.insert({LI->getPointerOperand(), LI->getType()});
    Loads.insert(LI);
    } else if (auto *SI = dyn_cast(&Inst)) {
  •  Pointers.insert({SI->getPointerOperand(),
    
  •                   SI->getValueOperand()->getType()});
    
  •  Pointers.insert(
    
  •      {SI->getPointerOperand(), SI->getValueOperand()->getType()});
     Stores.insert(SI);
    
    } else if (auto *CB = dyn_cast(&Inst))
    Calls.insert(CB);
    @@ -171,7 +172,8 @@ void AAEvaluator::runInternal(Function &F, AAResults &AA) {
    ++MayAliasCount;
    break;
    case AliasResult::PartialAlias:
  •      PrintLoadStoreResults(AR, PrintPartialAlias, Load, Store, F.getParent());
    
  •      PrintLoadStoreResults(AR, PrintPartialAlias, Load, Store,
    
  •                            F.getParent());
         ++PartialAliasCount;
         break;
       case AliasResult::MustAlias:
    

@@ -349,7 +351,7 @@ class AAEvalLegacyPass : public FunctionPass {
return false;
}
};
-}
+} // namespace llvm

char AAEvalLegacyPass::ID = 0;
INITIALIZE_PASS_BEGIN(AAEvalLegacyPass, "aa-eval",
diff --git a/llvm/lib/Analysis/AliasSetTracker.cpp b/llvm/lib/Analysis/AliasSetTracker.cpp
index 91b889116dfa2d3..8c65c001d2fe311 100644
--- a/llvm/lib/Analysis/AliasSetTracker.cpp
+++ b/llvm/lib/Analysis/AliasSetTracker.cpp
@@ -49,7 +49,7 @@ void AliasSet::mergeSetIn(AliasSet &AS, AliasSetTracker &AST,
bool WasMustAlias = (Alias == SetMustAlias);
// Update the alias and access types of this set...
Access |= AS.Access;

  • Alias |= AS.Alias;
  • Alias |= AS.Alias;

    if (Alias == SetMustAlias) {
    // Check that these two merged sets really are must aliases. Since both
    @@ -73,7 +73,7 @@ void AliasSet::mergeSetIn(AliasSet &AS, AliasSetTracker &AST,
    }

    bool ASHadUnknownInsts = !AS.UnknownInsts.empty();

  • if (UnknownInsts.empty()) { // Merge call sites...
  • if (UnknownInsts.empty()) { // Merge call sites...
    if (ASHadUnknownInsts) {
    std::swap(UnknownInsts, AS.UnknownInsts);
    addRef();
    @@ -107,8 +107,8 @@ void AliasSetTracker::removeAliasSet(AliasSet *AS) {
    Fwd->dropRef(*this);
    AS->Forward = nullptr;
    } else // Update TotalMayAliasSetSize only if not forwarding.
  •  if (AS->Alias == AliasSet::SetMayAlias)
    
  •    TotalMayAliasSetSize -= AS->size();
    
  • if (AS->Alias == AliasSet::SetMayAlias)

  •  TotalMayAliasSetSize -= AS->size();
    

    AliasSets.erase(AS);
    // If we've removed the saturated alias set, set saturated ma

@joker-eph
Copy link
Collaborator Author

@llvm/pr-subscribers-mlir-async

Test description...

Line 2

Patch is 4.74 MiB, truncated to 10kB below, full version: https://github.com/llvm/llvm-project/pull/65439.diff

Changes
diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index a453601d7ad93d1..3d25ce9488f126e 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -46,21 +46,33 @@ quality.
 Stay Informed
 -------------

-Developers should stay informed by reading the LLVM Discourse forums.
-If you are doing anything more than just casual work on LLVM, it is suggested that you also
-subscribe to the "commits" mailing list for the subproject you're interested in,
+Developers should stay informed by reading the LLVM Discourse forums
and subscribing
+to the categories of interest for notifications.
+
+Paying attention to changes being made by others is a good way to see what other people
+are interested in and watching the flow of the project as a whole.
+
+Contibutions to the project are made through :ref:GitHub Pull Requests <github-reviews>.
+You can subscribe to notification for areas of the codebase by joining
+one of the pr-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+GitHub teams. This mapping <https://github.com/llvm/llvm-project/blob/main/.github/new-prs-labeler.yml>_
+indicates which team is associated with a particular paths in the repository.
+
+You can also subscribe to the "commits" mailing list for a subproject you're interested in,
such as llvm-commits <http://lists.llvm.org/mailman/listinfo/llvm-commits>, cfe-commits <http://lists.llvm.org/mailman/listinfo/cfe-commits>, or lldb-commits -<http://lists.llvm.org/mailman/listinfo/lldb-commits>_. Reading the
-"commits" list and paying attention to changes being made by others is a good
-way to see what other people are interested in and watching the flow of the
-project as a whole.

-We recommend that active developers monitor incoming issues to our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>_ and preferably subscribe to the llvm-bugs +<http://lists.llvm.org/mailman/listinfo/lldb-commits>.
+
+Missing features and bugs are tracked through our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>

+and assigned labels. We recommend that active developers monitor incoming issues.
+You can subscribe for notification for specific components by joining
+one of the issue-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+teams.
+You may also subscribe to the llvm-bugs <http://lists.llvm.org/mailman/listinfo/llvm-bugs>_ email list to keep track
-of bugs and enhancements occurring in LLVM. We really appreciate people who are
-proactive at catching incoming bugs in their components and dealing with them
+of bugs and enhancements occurring in the entire project. We really appreciate people
+who are proactive at catching incoming bugs in their components and dealing with them
promptly.

Please be aware that all public LLVM mailing lists and discourse forums are public and archived, and
diff --git a/llvm/lib/Analysis/AliasAnalysis.cpp b/llvm/lib/Analysis/AliasAnalysis.cpp
index 75fe68a80ab6085..579087857b7eb0e 100644
--- a/llvm/lib/Analysis/AliasAnalysis.cpp
+++ b/llvm/lib/Analysis/AliasAnalysis.cpp
@@ -56,8 +56,8 @@

using namespace llvm;

-STATISTIC(NumNoAlias, "Number of NoAlias results");
-STATISTIC(NumMayAlias, "Number of MayAlias results");
+STATISTIC(NumNoAlias, "Number of NoAlias results");
+STATISTIC(NumMayAlias, "Number of MayAlias results");
STATISTIC(NumMustAlias, "Number of MustAlias results");

namespace llvm {
@@ -116,8 +116,8 @@ AliasResult AAResults::alias(const MemoryLocation &LocA,
if (EnableAATrace) {
for (unsigned I = 0; I < AAQI.Depth; ++I)
dbgs() << " ";

  • dbgs() << "Start " << *LocA.Ptr << " @ " << LocA.Size << ", "
  •       << *LocB.Ptr << " @ " << LocB.Size << "\n";
    
  • dbgs() << "Start " << *LocA.Ptr << " @ " << LocA.Size << ", " << *LocB.Ptr

  •       << " @ " << LocB.Size << "\n";
    

    }

    AAQI.Depth++;
    @@ -131,8 +131,8 @@ AliasResult AAResults::alias(const MemoryLocation &LocA,
    if (EnableAATrace) {
    for (unsigned I = 0; I < AAQI.Depth; ++I)
    dbgs() << " ";

  • dbgs() << "End " << *LocA.Ptr << " @ " << LocA.Size << ", "
  •       << *LocB.Ptr << " @ " << LocB.Size << " = " << Result << "\n";
    
  • dbgs() << "End " << *LocA.Ptr << " @ " << LocA.Size << ", " << *LocB.Ptr

  •       << " @ " << LocB.Size << " = " << Result << "\n";
    

    }

    if (AAQI.Depth == 0) {
    @@ -584,7 +584,8 @@ ModRefInfo AAResults::getModRefInfo(const AtomicCmpXchgInst *CX,
    ModRefInfo AAResults::getModRefInfo(const AtomicRMWInst *RMW,
    const MemoryLocation &Loc,
    AAQueryInfo &AAQI) {

  • // Acquire/Release atomicrmw has properties that matter for arbitrary addresses.
  • // Acquire/Release atomicrmw has properties that matter for arbitrary
  • // addresses.
    if (isStrongerThanMonotonic(RMW->getOrdering()))
    return ModRefInfo::ModRef;

@@ -646,8 +647,7 @@ ModRefInfo AAResults::getModRefInfo(const Instruction *I,
/// with a smarter AA in place, this test is just wasting compile time.
ModRefInfo AAResults::callCapturesBefore(const Instruction *I,
const MemoryLocation &MemLoc,

  •                                     DominatorTree *DT,
    
  •                                     AAQueryInfo &AAQI) {
    
  •                                     DominatorTree *DT, AAQueryInfo &AAQI) {
    
    if (!DT)
    return ModRefInfo::ModRef;

@@ -718,7 +718,7 @@ bool AAResults::canInstructionRangeModRef(const Instruction &I1,
"Instructions not in same basic block!");
BasicBlock::const_iterator I = I1.getIterator();
BasicBlock::const_iterator E = I2.getIterator();

  • ++E; // Convert from inclusive to exclusive range.
  • ++E; // Convert from inclusive to exclusive range.

    for (; I != E; ++I) // Check every instruction in range
    if (isModOrRefSet(getModRefInfo(&*I, Loc) & Mode))
    diff --git a/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp b/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    index a551ea6b69c5a40..9811af81642db4f 100644
    --- a/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    +++ b/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    @@ -24,7 +24,8 @@ static cl::opt PrintAll("print-all-alias-modref-info", cl::ReallyHidden);

static cl::opt PrintNoAlias("print-no-aliases", cl::ReallyHidden);
static cl::opt PrintMayAlias("print-may-aliases", cl::ReallyHidden);
-static cl::opt PrintPartialAlias("print-partial-aliases", cl::ReallyHidden);
+static cl::opt PrintPartialAlias("print-partial-aliases",

  •                                   cl::ReallyHidden);
    

static cl::opt PrintMustAlias("print-must-aliases", cl::ReallyHidden);

static cl::opt PrintNoModRef("print-no-modref", cl::ReallyHidden);
@@ -68,9 +69,9 @@ static void PrintResults(AliasResult AR, bool P,
}
}

-static inline void PrintModRefResults(

  • const char *Msg, bool P, Instruction *I,
  • std::pair<const Value *, Type *> Loc, Module *M) {
    +static inline void PrintModRefResults(const char *Msg, bool P, Instruction *I,
  •                                  std::pair<const Value *, Type *> Loc,
    
  •                                  Module *M) {
    
    if (PrintAll || P) {
    errs() << " " << Msg << ": Ptr: ";
    Loc.second->print(errs(), false, /* NoDetails */ true);
    @@ -115,8 +116,8 @@ void AAEvaluator::runInternal(Function &F, AAResults &AA) {
    Pointers.insert({LI->getPointerOperand(), LI->getType()});
    Loads.insert(LI);
    } else if (auto *SI = dyn_cast(&Inst)) {
  •  Pointers.insert({SI->getPointerOperand(),
    
  •                   SI->getValueOperand()->getType()});
    
  •  Pointers.insert(
    
  •      {SI->getPointerOperand(), SI->getValueOperand()->getType()});
     Stores.insert(SI);
    
    } else if (auto *CB = dyn_cast(&Inst))
    Calls.insert(CB);
    @@ -171,7 +172,8 @@ void AAEvaluator::runInternal(Function &F, AAResults &AA) {
    ++MayAliasCount;
    break;
    case AliasResult::PartialAlias:
  •      PrintLoadStoreResults(AR, PrintPartialAlias, Load, Store, F.getParent());
    
  •      PrintLoadStoreResults(AR, PrintPartialAlias, Load, Store,
    
  •                            F.getParent());
         ++PartialAliasCount;
         break;
       case AliasResult::MustAlias:
    

@@ -349,7 +351,7 @@ class AAEvalLegacyPass : public FunctionPass {
return false;
}
};
-}
+} // namespace llvm

char AAEvalLegacyPass::ID = 0;
INITIALIZE_PASS_BEGIN(AAEvalLegacyPass, "aa-eval",
diff --git a/llvm/lib/Analysis/AliasSetTracker.cpp b/llvm/lib/Analysis/AliasSetTracker.cpp
index 91b889116dfa2d3..8c65c001d2fe311 100644
--- a/llvm/lib/Analysis/AliasSetTracker.cpp
+++ b/llvm/lib/Analysis/AliasSetTracker.cpp
@@ -49,7 +49,7 @@ void AliasSet::mergeSetIn(AliasSet &AS, AliasSetTracker &AST,
bool WasMustAlias = (Alias == SetMustAlias);
// Update the alias and access types of this set...
Access |= AS.Access;

  • Alias |= AS.Alias;
  • Alias |= AS.Alias;

    if (Alias == SetMustAlias) {
    // Check that these two merged sets really are must aliases. Since both
    @@ -73,7 +73,7 @@ void AliasSet::mergeSetIn(AliasSet &AS, AliasSetTracker &AST,
    }

    bool ASHadUnknownInsts = !AS.UnknownInsts.empty();

  • if (UnknownInsts.empty()) { // Merge call sites...
  • if (UnknownInsts.empty()) { // Merge call sites...
    if (ASHadUnknownInsts) {
    std::swap(UnknownInsts, AS.UnknownInsts);
    addRef();
    @@ -107,8 +107,8 @@ void AliasSetTracker::removeAliasSet(AliasSet *AS) {
    Fwd->dropRef(*this);
    AS->Forward = nullptr;
    } else // Update TotalMayAliasSetSize only if not forwarding.
  •  if (AS->Alias == AliasSet::SetMayAlias)
    
  •    TotalMayAliasSetSize -= AS->size();
    
  • if (AS->Alias == AliasSet::SetMayAlias)

  •  TotalMayAliasSetSize -= AS->size();
    

    AliasSets.erase(AS);
    // If we've removed the saturated alias set, set saturated ma

1 similar comment
@joker-eph
Copy link
Collaborator Author

@llvm/pr-subscribers-mlir-async

Test description...

Line 2

Patch is 4.74 MiB, truncated to 10kB below, full version: https://github.com/llvm/llvm-project/pull/65439.diff

Changes
diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index a453601d7ad93d1..3d25ce9488f126e 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -46,21 +46,33 @@ quality.
 Stay Informed
 -------------

-Developers should stay informed by reading the LLVM Discourse forums.
-If you are doing anything more than just casual work on LLVM, it is suggested that you also
-subscribe to the "commits" mailing list for the subproject you're interested in,
+Developers should stay informed by reading the LLVM Discourse forums
and subscribing
+to the categories of interest for notifications.
+
+Paying attention to changes being made by others is a good way to see what other people
+are interested in and watching the flow of the project as a whole.
+
+Contibutions to the project are made through :ref:GitHub Pull Requests <github-reviews>.
+You can subscribe to notification for areas of the codebase by joining
+one of the pr-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+GitHub teams. This mapping <https://github.com/llvm/llvm-project/blob/main/.github/new-prs-labeler.yml>_
+indicates which team is associated with a particular paths in the repository.
+
+You can also subscribe to the "commits" mailing list for a subproject you're interested in,
such as llvm-commits <http://lists.llvm.org/mailman/listinfo/llvm-commits>, cfe-commits <http://lists.llvm.org/mailman/listinfo/cfe-commits>, or lldb-commits -<http://lists.llvm.org/mailman/listinfo/lldb-commits>_. Reading the
-"commits" list and paying attention to changes being made by others is a good
-way to see what other people are interested in and watching the flow of the
-project as a whole.

-We recommend that active developers monitor incoming issues to our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>_ and preferably subscribe to the llvm-bugs +<http://lists.llvm.org/mailman/listinfo/lldb-commits>.
+
+Missing features and bugs are tracked through our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>

+and assigned labels. We recommend that active developers monitor incoming issues.
+You can subscribe for notification for specific components by joining
+one of the issue-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+teams.
+You may also subscribe to the llvm-bugs <http://lists.llvm.org/mailman/listinfo/llvm-bugs>_ email list to keep track
-of bugs and enhancements occurring in LLVM. We really appreciate people who are
-proactive at catching incoming bugs in their components and dealing with them
+of bugs and enhancements occurring in the entire project. We really appreciate people
+who are proactive at catching incoming bugs in their components and dealing with them
promptly.

Please be aware that all public LLVM mailing lists and discourse forums are public and archived, and
diff --git a/llvm/lib/Analysis/AliasAnalysis.cpp b/llvm/lib/Analysis/AliasAnalysis.cpp
index 75fe68a80ab6085..579087857b7eb0e 100644
--- a/llvm/lib/Analysis/AliasAnalysis.cpp
+++ b/llvm/lib/Analysis/AliasAnalysis.cpp
@@ -56,8 +56,8 @@

using namespace llvm;

-STATISTIC(NumNoAlias, "Number of NoAlias results");
-STATISTIC(NumMayAlias, "Number of MayAlias results");
+STATISTIC(NumNoAlias, "Number of NoAlias results");
+STATISTIC(NumMayAlias, "Number of MayAlias results");
STATISTIC(NumMustAlias, "Number of MustAlias results");

namespace llvm {
@@ -116,8 +116,8 @@ AliasResult AAResults::alias(const MemoryLocation &LocA,
if (EnableAATrace) {
for (unsigned I = 0; I < AAQI.Depth; ++I)
dbgs() << " ";

  • dbgs() << "Start " << *LocA.Ptr << " @ " << LocA.Size << ", "
  •       << *LocB.Ptr << " @ " << LocB.Size << "\n";
    
  • dbgs() << "Start " << *LocA.Ptr << " @ " << LocA.Size << ", " << *LocB.Ptr

  •       << " @ " << LocB.Size << "\n";
    

    }

    AAQI.Depth++;
    @@ -131,8 +131,8 @@ AliasResult AAResults::alias(const MemoryLocation &LocA,
    if (EnableAATrace) {
    for (unsigned I = 0; I < AAQI.Depth; ++I)
    dbgs() << " ";

  • dbgs() << "End " << *LocA.Ptr << " @ " << LocA.Size << ", "
  •       << *LocB.Ptr << " @ " << LocB.Size << " = " << Result << "\n";
    
  • dbgs() << "End " << *LocA.Ptr << " @ " << LocA.Size << ", " << *LocB.Ptr

  •       << " @ " << LocB.Size << " = " << Result << "\n";
    

    }

    if (AAQI.Depth == 0) {
    @@ -584,7 +584,8 @@ ModRefInfo AAResults::getModRefInfo(const AtomicCmpXchgInst *CX,
    ModRefInfo AAResults::getModRefInfo(const AtomicRMWInst *RMW,
    const MemoryLocation &Loc,
    AAQueryInfo &AAQI) {

  • // Acquire/Release atomicrmw has properties that matter for arbitrary addresses.
  • // Acquire/Release atomicrmw has properties that matter for arbitrary
  • // addresses.
    if (isStrongerThanMonotonic(RMW->getOrdering()))
    return ModRefInfo::ModRef;

@@ -646,8 +647,7 @@ ModRefInfo AAResults::getModRefInfo(const Instruction *I,
/// with a smarter AA in place, this test is just wasting compile time.
ModRefInfo AAResults::callCapturesBefore(const Instruction *I,
const MemoryLocation &MemLoc,

  •                                     DominatorTree *DT,
    
  •                                     AAQueryInfo &AAQI) {
    
  •                                     DominatorTree *DT, AAQueryInfo &AAQI) {
    
    if (!DT)
    return ModRefInfo::ModRef;

@@ -718,7 +718,7 @@ bool AAResults::canInstructionRangeModRef(const Instruction &I1,
"Instructions not in same basic block!");
BasicBlock::const_iterator I = I1.getIterator();
BasicBlock::const_iterator E = I2.getIterator();

  • ++E; // Convert from inclusive to exclusive range.
  • ++E; // Convert from inclusive to exclusive range.

    for (; I != E; ++I) // Check every instruction in range
    if (isModOrRefSet(getModRefInfo(&*I, Loc) & Mode))
    diff --git a/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp b/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    index a551ea6b69c5a40..9811af81642db4f 100644
    --- a/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    +++ b/llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
    @@ -24,7 +24,8 @@ static cl::opt PrintAll("print-all-alias-modref-info", cl::ReallyHidden);

static cl::opt PrintNoAlias("print-no-aliases", cl::ReallyHidden);
static cl::opt PrintMayAlias("print-may-aliases", cl::ReallyHidden);
-static cl::opt PrintPartialAlias("print-partial-aliases", cl::ReallyHidden);
+static cl::opt PrintPartialAlias("print-partial-aliases",

  •                                   cl::ReallyHidden);
    

static cl::opt PrintMustAlias("print-must-aliases", cl::ReallyHidden);

static cl::opt PrintNoModRef("print-no-modref", cl::ReallyHidden);
@@ -68,9 +69,9 @@ static void PrintResults(AliasResult AR, bool P,
}
}

-static inline void PrintModRefResults(

  • const char *Msg, bool P, Instruction *I,
  • std::pair<const Value *, Type *> Loc, Module *M) {
    +static inline void PrintModRefResults(const char *Msg, bool P, Instruction *I,
  •                                  std::pair<const Value *, Type *> Loc,
    
  •                                  Module *M) {
    
    if (PrintAll || P) {
    errs() << " " << Msg << ": Ptr: ";
    Loc.second->print(errs(), false, /* NoDetails */ true);
    @@ -115,8 +116,8 @@ void AAEvaluator::runInternal(Function &F, AAResults &AA) {
    Pointers.insert({LI->getPointerOperand(), LI->getType()});
    Loads.insert(LI);
    } else if (auto *SI = dyn_cast(&Inst)) {
  •  Pointers.insert({SI->getPointerOperand(),
    
  •                   SI->getValueOperand()->getType()});
    
  •  Pointers.insert(
    
  •      {SI->getPointerOperand(), SI->getValueOperand()->getType()});
     Stores.insert(SI);
    
    } else if (auto *CB = dyn_cast(&Inst))
    Calls.insert(CB);
    @@ -171,7 +172,8 @@ void AAEvaluator::runInternal(Function &F, AAResults &AA) {
    ++MayAliasCount;
    break;
    case AliasResult::PartialAlias:
  •      PrintLoadStoreResults(AR, PrintPartialAlias, Load, Store, F.getParent());
    
  •      PrintLoadStoreResults(AR, PrintPartialAlias, Load, Store,
    
  •                            F.getParent());
         ++PartialAliasCount;
         break;
       case AliasResult::MustAlias:
    

@@ -349,7 +351,7 @@ class AAEvalLegacyPass : public FunctionPass {
return false;
}
};
-}
+} // namespace llvm

char AAEvalLegacyPass::ID = 0;
INITIALIZE_PASS_BEGIN(AAEvalLegacyPass, "aa-eval",
diff --git a/llvm/lib/Analysis/AliasSetTracker.cpp b/llvm/lib/Analysis/AliasSetTracker.cpp
index 91b889116dfa2d3..8c65c001d2fe311 100644
--- a/llvm/lib/Analysis/AliasSetTracker.cpp
+++ b/llvm/lib/Analysis/AliasSetTracker.cpp
@@ -49,7 +49,7 @@ void AliasSet::mergeSetIn(AliasSet &AS, AliasSetTracker &AST,
bool WasMustAlias = (Alias == SetMustAlias);
// Update the alias and access types of this set...
Access |= AS.Access;

  • Alias |= AS.Alias;
  • Alias |= AS.Alias;

    if (Alias == SetMustAlias) {
    // Check that these two merged sets really are must aliases. Since both
    @@ -73,7 +73,7 @@ void AliasSet::mergeSetIn(AliasSet &AS, AliasSetTracker &AST,
    }

    bool ASHadUnknownInsts = !AS.UnknownInsts.empty();

  • if (UnknownInsts.empty()) { // Merge call sites...
  • if (UnknownInsts.empty()) { // Merge call sites...
    if (ASHadUnknownInsts) {
    std::swap(UnknownInsts, AS.UnknownInsts);
    addRef();
    @@ -107,8 +107,8 @@ void AliasSetTracker::removeAliasSet(AliasSet *AS) {
    Fwd->dropRef(*this);
    AS->Forward = nullptr;
    } else // Update TotalMayAliasSetSize only if not forwarding.
  •  if (AS->Alias == AliasSet::SetMayAlias)
    
  •    TotalMayAliasSetSize -= AS->size();
    
  • if (AS->Alias == AliasSet::SetMayAlias)

  •  TotalMayAliasSetSize -= AS->size();
    

    AliasSets.erase(AS);
    // If we've removed the saturated alias set, set saturated ma

@joker-eph
Copy link
Collaborator Author

@llvm/pr-subscribers-mlir-async

Test description...

Line 2

Full diff: https://github.com/llvm/llvm-project/pull/65439.diff

Changes
13 Files:
  • (modified) llvm/docs/DeveloperPolicy.rst (+23-11)
  • (modified) llvm/lib/FileCheck/CMakeLists.txt (+4-6)
  • (modified) llvm/lib/FileCheck/FileCheck.cpp (+13-11)
  • (modified) llvm/lib/Frontend/CMakeLists.txt (+1-3)
  • (modified) llvm/lib/Frontend/HLSL/CMakeLists.txt (+7-11)
  • (modified) llvm/lib/Frontend/OpenACC/CMakeLists.txt (+7-10)
  • (modified) llvm/lib/Frontend/OpenMP/CMakeLists.txt (+8-20)
  • (modified) llvm/lib/FuzzMutate/CMakeLists.txt (+17-31)
  • (modified) llvm/lib/Fuzzer/README.txt (+1-1)
  • (modified) llvm/lib/InterfaceStub/CMakeLists.txt (+3-11)
  • (modified) llvm/lib/MCA/HardwareUnits/LSUnit.cpp (+5-7)
  • (modified) llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp (+5-3)
  • (modified) llvm/lib/MCA/Stages/InstructionTables.cpp (+1-2)

diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index a453601d7ad93d1..3d25ce9488f126e 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -46,21 +46,33 @@ quality.
Stay Informed

-Developers should stay informed by reading the LLVM Discourse forums.
-If you are doing anything more than just casual work on LLVM, it is suggested that you also
-subscribe to the "commits" mailing list for the subproject you're interested in,
+Developers should stay informed by reading the LLVM Discourse forums
and subscribing
+to the categories of interest for notifications.
+
+Paying attention to changes being made by others is a good way to see what other people
+are interested in and watching the flow of the project as a whole.
+
+Contibutions to the project are made through :ref:GitHub Pull Requests <github-reviews>.
+You can subscribe to notification for areas of the codebase by joining
+one of the pr-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+GitHub teams. This mapping <https://github.com/llvm/llvm-project/blob/main/.github/new-prs-labeler.yml>_
+indicates which team is associated with a particular paths in the repository.
+
+You can also subscribe to the "commits" mailing list for a subproject you're interested in,
such as llvm-commits <http://lists.llvm.org/mailman/listinfo/llvm-commits>, cfe-commits <http://lists.llvm.org/mailman/listinfo/cfe-commits>, or lldb-commits -<http://lists.llvm.org/mailman/listinfo/lldb-commits>_. Reading the
-"commits" list and paying attention to changes being made by others is a good
-way to see what other people are interested in and watching the flow of the
-project as a whole.

-We recommend that active developers monitor incoming issues to our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>_ and preferably subscribe to the llvm-bugs +<http://lists.llvm.org/mailman/listinfo/lldb-commits>.
+
+Missing features and bugs are tracked through our GitHub issue tracker <https://github.com/llvm/llvm-project/issues>

+and assigned labels. We recommend that active developers monitor incoming issues.
+You can subscribe for notification for specific components by joining
+one of the issue-subscribers-* <https://github.com/orgs/llvm/teams?query=issue-subscribers>_
+teams.
+You may also subscribe to the llvm-bugs <http://lists.llvm.org/mailman/listinfo/llvm-bugs>_ email list to keep track
-of bugs and enhancements occurring in LLVM. We really appreciate people who are
-proactive at catching incoming bugs in their components and dealing with them
+of bugs and enhancements occurring in the entire project. We really appreciate people
+who are proactive at catching incoming bugs in their components and dealing with them
promptly.

Please be aware that all public LLVM mailing lists and discourse forums are public and archived, and
diff --git a/llvm/lib/FileCheck/CMakeLists.txt b/llvm/lib/FileCheck/CMakeLists.txt
index 91c80e1482f1985..bbc323b1f5c0449 100644
--- a/llvm/lib/FileCheck/CMakeLists.txt
+++ b/llvm/lib/FileCheck/CMakeLists.txt
@@ -1,8 +1,6 @@
-add_llvm_component_library(LLVMFileCheck

  • FileCheck.cpp
    +add_llvm_component_library(LLVMFileCheck FileCheck.cpp

  • ADDITIONAL_HEADER_DIRS

  • "${LLVM_MAIN_INCLUDE_DIR}/llvm/FileCheck"
    -)

  •                           ADDITIONAL_HEADER_DIRS
    
  •                       "${LLVM_MAIN_INCLUDE_DIR}/llvm/FileCheck")
    

-target_link_libraries(LLVMFileCheck LLVMSupport)

  • target_link_libraries(LLVMFileCheck LLVMSupport)
    diff --git a/llvm/lib/FileCheck/FileCheck.cpp b/llvm/lib/FileCheck/FileCheck.cpp
    index 49fda8fb63cd36c..129699169d84014 100644
    --- a/llvm/lib/FileCheck/FileCheck.cpp
    +++ b/llvm/lib/FileCheck/FileCheck.cpp
    @@ -9,8 +9,8 @@
    // FileCheck does a line-by line check of a file that validates whether it
    // contains the expected content. This is useful for regression tests etc.
    //
    -// This file implements most of the API that will be used by the FileCheck utility
    -// as well as various unittests.
    +// This file implements most of the API that will be used by the FileCheck
    +// utility as well as various unittests.
    //===----------------------------------------------------------------------===//

#include "llvm/FileCheck/FileCheck.h"
@@ -307,7 +307,7 @@ Pattern::parseVariable(StringRef &Str, const SourceMgr &SM) {

StringRef Name = Str.take_front(I);
Str = Str.substr(I);

  • return VariableProperties {Name, IsPseudo};
  • return VariableProperties{Name, IsPseudo};
    }

// StringRef holding all characters considered as horizontal whitespaces by
@@ -1847,20 +1847,22 @@ bool FileCheck::readCheckFile(

 // Verify that CHECK-LABEL lines do not define or use variables
 if ((CheckTy == Check::CheckLabel) && P.hasVariable()) {
  •  SM.PrintMessage(
    
  •      SMLoc::getFromPointer(UsedPrefixStart), SourceMgr::DK_Error,
    
  •      "found '" + UsedPrefix + "-LABEL:'"
    
  •                               " with variable definition or use");
    
  •  SM.PrintMessage(SMLoc::getFromPointer(UsedPrefixStart),
    
  •                  SourceMgr::DK_Error,
    
  •                  "found '" + UsedPrefix +
    
  •                      "-LABEL:'"
    
  •                      " with variable definition or use");
     return true;
    
    }
  • // Verify that CHECK-NEXT/SAME/EMPTY lines have at least one CHECK line before them.
  • // Verify that CHECK-NEXT/SAME/EMPTY lines have at least one CHECK line
  • // before them.
    if ((CheckTy == Check::CheckNext || CheckTy == Check::CheckSame ||
    CheckTy == Check::CheckEmpty) &&
    CheckStrings->empty()) {
  •  StringRef Type = CheckTy == Check::CheckNext
    
  •                       ? "NEXT"
    
  •                       : CheckTy == Check::CheckEmpty ? "EMPTY" : "SAME";
    
  •  StringRef Type = CheckTy == Check::CheckNext    ? "NEXT"
    
  •                   : CheckTy == Check::CheckEmpty ? "EMPTY"
    
  •                                                  : "SAME";
     SM.PrintMessage(SMLoc::getFromPointer(UsedPrefixStart),
                     SourceMgr::DK_Error,
                     "found '" + UsedPrefix + "-" + Type +
    

diff --git a/llvm/lib/Frontend/CMakeLists.txt b/llvm/lib/Frontend/CMakeLists.txt
index fa48c975a8b3e5a..231726c49590ee6 100644
--- a/llvm/lib/Frontend/CMakeLists.txt
+++ b/llvm/lib/Frontend/CMakeLists.txt
@@ -1,3 +1 @@
-add_subdirectory(HLSL)
-add_subdirectory(OpenACC)
-add_subdirectory(OpenMP)
+add_subdirectory(HLSL) add_subdirectory(OpenACC) add_subdirectory(OpenMP)
diff --git a/llvm/lib/Frontend/HLSL/CMakeLists.txt b/llvm/lib/Frontend/HLSL/CMakeLists.txt
index eda6cb8e69a4971..980d9118acdcf48 100644
--- a/llvm/lib/Frontend/HLSL/CMakeLists.txt
+++ b/llvm/lib/Frontend/HLSL/CMakeLists.txt
@@ -1,14 +1,10 @@
-add_llvm_component_library(LLVMFrontendHLSL

  • HLSLResource.cpp
    +add_llvm_component_library(LLVMFrontendHLSL HLSLResource.cpp

  • ADDITIONAL_HEADER_DIRS

  • ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend

  • ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend/HLSL

  •                           ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
    
  •                       llvm / Frontend ${LLVM_MAIN_INCLUDE_DIR} / llvm /
    
  •                       Frontend /
    
  •                       HLSL
    
  • DEPENDS
  • intrinsics_gen
  •                           DEPENDS intrinsics_gen
    
  • LINK_COMPONENTS
  • Core
  • Support
  • )
  •                               LINK_COMPONENTS Core Support)
    

diff --git a/llvm/lib/Frontend/OpenACC/CMakeLists.txt b/llvm/lib/Frontend/OpenACC/CMakeLists.txt
index f35201497869058..348d07347c79018 100644
--- a/llvm/lib/Frontend/OpenACC/CMakeLists.txt
+++ b/llvm/lib/Frontend/OpenACC/CMakeLists.txt
@@ -1,13 +1,10 @@
-add_llvm_component_library(LLVMFrontendOpenACC

  • ACC.cpp
    +add_llvm_component_library(LLVMFrontendOpenACC ACC.cpp

  • ADDITIONAL_HEADER_DIRS

  • ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend

  • ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend/OpenACC

  •                           ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
    
  •                       llvm / Frontend ${LLVM_MAIN_INCLUDE_DIR} / llvm /
    
  •                       Frontend /
    
  •                       OpenACC
    
  • DEPENDS
  • acc_gen
    -)

-target_link_libraries(LLVMFrontendOpenACC LLVMSupport)

  •                           DEPENDS acc_gen)
    
  • target_link_libraries(LLVMFrontendOpenACC LLVMSupport)
    diff --git a/llvm/lib/Frontend/OpenMP/CMakeLists.txt b/llvm/lib/Frontend/OpenMP/CMakeLists.txt
    index a2eedabc3ed6903..35f7192c5fe9553 100644
    --- a/llvm/lib/Frontend/OpenMP/CMakeLists.txt
    +++ b/llvm/lib/Frontend/OpenMP/CMakeLists.txt
    @@ -1,23 +1,11 @@
    -add_llvm_component_library(LLVMFrontendOpenMP

  • OMP.cpp
  • OMPContext.cpp
  • OMPIRBuilder.cpp
    +add_llvm_component_library(
  • LLVMFrontendOpenMP OMP.cpp OMPContext.cpp OMPIRBuilder.cpp
  • ADDITIONAL_HEADER_DIRS
  • ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend
  • ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend/OpenMP
  •    ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
    
  • llvm / Frontend ${LLVM_MAIN_INCLUDE_DIR} / llvm / Frontend /
  • OpenMP
  • DEPENDS
  • intrinsics_gen
  • omp_gen
  •    DEPENDS intrinsics_gen omp_gen
    
  • LINK_COMPONENTS
  • Core
  • Support
  • TargetParser
  • TransformUtils
  • Analysis
  • MC
  • Scalar
  • BitReader
  • )
  •        LINK_COMPONENTS Core Support TargetParser TransformUtils Analysis MC
    
  •            Scalar BitReader)
    

diff --git a/llvm/lib/FuzzMutate/CMakeLists.txt b/llvm/lib/FuzzMutate/CMakeLists.txt
index e162c1bbe8e1078..4fa1aa2bed6eb96 100644
--- a/llvm/lib/FuzzMutate/CMakeLists.txt
+++ b/llvm/lib/FuzzMutate/CMakeLists.txt
@@ -1,37 +1,23 @@
-# Generic helper for fuzzer binaries.
-# This should not depend on LLVM IR etc.
-add_llvm_component_library(LLVMFuzzerCLI

  • FuzzerCLI.cpp

  • PARTIAL_SOURCES_INTENDED
    +#Generic helper for fuzzer binaries.
    +#This should not depend on LLVM IR etc.
    +add_llvm_component_library(LLVMFuzzerCLI FuzzerCLI.cpp PARTIAL_SOURCES_INTENDED

  • ADDITIONAL_HEADER_DIRS

  • ${LLVM_MAIN_INCLUDE_DIR}/llvm/FuzzMutate

  •                           ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
    
  •                       llvm /
    
  •                       FuzzMutate
    
  • LINK_COMPONENTS
  • Support
  • TargetParser
  • )
  •                           LINK_COMPONENTS Support TargetParser)
    

-# Library for using LLVM IR together with fuzzers.
-add_llvm_component_library(LLVMFuzzMutate

  • IRMutator.cpp
  • OpDescriptor.cpp
  • Operations.cpp
  • RandomIRBuilder.cpp
    +#Library for using LLVM IR together with fuzzers.
  • add_llvm_component_library(
  •    LLVMFuzzMutate IRMutator.cpp OpDescriptor.cpp Operations
    
  •        .cpp RandomIRBuilder.cpp
    
  • ADDITIONAL_HEADER_DIRS
  • ${LLVM_MAIN_INCLUDE_DIR}/llvm/FuzzMutate
  •            ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
    
  •    llvm /
    
  •    FuzzMutate
    
  • DEPENDS
  • intrinsics_gen
  •        DEPENDS intrinsics_gen
    
  • LINK_COMPONENTS
  • Analysis
  • BitReader
  • BitWriter
  • Core
  • Scalar
  • Support
  • TargetParser
  • TransformUtils
  • )
  •            LINK_COMPONENTS Analysis BitReader BitWriter Core Scalar Support
    
  •                TargetParser TransformUtils)
    

diff --git a/llvm/lib/Fuzzer/README.txt b/llvm/lib/Fuzzer/README.txt
index 53ac637638f64ea..de140b0406de1f2 100644
--- a/llvm/lib/Fuzzer/README.txt
+++ b/llvm/lib/Fuzzer/README.txt
@@ -1 +1 @@
-libFuzzer was moved to compiler-rt in https://reviews.llvm.org/D36908.
+libFuzzer was moved to compiler - rt in https: // reviews.llvm.org/D36908.
diff --git a/llvm/lib/InterfaceStub/CMakeLists.txt b/llvm/lib/InterfaceStub/CMakeLists.txt
index 68fe067701f23f4..c5997d1922f5a80 100644
--- a/llvm/lib/InterfaceStub/CMakeLists.txt
+++ b/llvm/lib/InterfaceStub/CMakeLists.txt
@@ -1,12 +1,4 @@
-add_llvm_component_library(LLVMInterfaceStub

  • ELFObjHandler.cpp
  • IFSHandler.cpp
  • IFSStub.cpp
    +add_llvm_component_library(
  • LLVMInterfaceStub ELFObjHandler.cpp IFSHandler.cpp IFSStub.cpp
  • LINK_COMPONENTS
  • BinaryFormat
  • MC
  • Object
  • Support
  • TargetParser
    -)
  •    LINK_COMPONENTS BinaryFormat MC Object Support TargetParser)
    

diff --git a/llvm/lib/MCA/HardwareUnits/LSUnit.cpp b/llvm/lib/MCA/HardwareUnits/LSUnit.cpp
index bdc8b3d0e390e21..3b54cc2c8fddf73 100644
--- a/llvm/lib/MCA/HardwareUnits/LSUnit.cpp
+++ b/llvm/lib/MCA/HardwareUnits/LSUnit.cpp
@@ -96,8 +96,8 @@ unsigned LSUnit::dispatch(const InstRef &IR) {
if (CurrentStoreBarrierGroupID) {
MemoryGroup &StoreGroup = getGroup(CurrentStoreBarrierGroupID);
LLVM_DEBUG(dbgs() << "[LSUnit]: GROUP DEP: ("

  •                    << CurrentStoreBarrierGroupID
    
  •                    << ") --> (" << NewGID << ")\n");
    
  •                    << CurrentStoreBarrierGroupID << ") --> (" << NewGID
    
  •                    << ")\n");
     StoreGroup.addSuccessor(&NewGroup, true);
    
    }

@@ -110,7 +110,6 @@ unsigned LSUnit::dispatch(const InstRef &IR) {
StoreGroup.addSuccessor(&NewGroup, !assumeNoAlias());
}

  • CurrentStoreGroupID = NewGID;
    if (IsStoreBarrier)
    CurrentStoreBarrierGroupID = NewGID;
    @@ -165,8 +164,7 @@ unsigned LSUnit::dispatch(const InstRef &IR) {
    if (IsLoadBarrier) {
    if (ImmediateLoadDominator) {
    MemoryGroup &LoadGroup = getGroup(ImmediateLoadDominator);
  •    LLVM_DEBUG(dbgs() << "[LSUnit]: GROUP DEP: ("
    
  •                      << ImmediateLoadDominator
    
  •    LLVM_DEBUG(dbgs() << "[LSUnit]: GROUP DEP: (" << ImmediateLoadDominator
                         << ") --> (" << NewGID << ")\n");
       LoadGroup.addSuccessor(&NewGroup, true);
     }
    

@@ -175,8 +173,8 @@ unsigned LSUnit::dispatch(const InstRef &IR) {
if (CurrentLoadBarrierGroupID) {
MemoryGroup &LoadGroup = getGroup(CurrentLoadBarrierGroupID);
LLVM_DEBUG(dbgs() << "[LSUnit]: GROUP DEP: ("

  •                      << CurrentLoadBarrierGroupID
    
  •                      << ") --> (" << NewGID << ")\n");
    
  •                      << CurrentLoadBarrierGroupID << ") --> (" << NewGID
    
  •                      << ")\n");
       LoadGroup.addSuccessor(&NewGroup, true);
     }
    
    }
    diff --git a/llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp b/llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp
    index 9297f0c4fd7bf1b..78ec5d06e8fedf9 100644
    --- a/llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp
    +++ b/llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp
    @@ -66,7 +66,8 @@ const RetireControlUnit::RUToken &RetireControlUnit::getCurrentToken() const {

unsigned RetireControlUnit::computeNextSlotIdx() const {
const RetireControlUnit::RUToken &Current = getCurrentToken();

  • unsigned NextSlotIdx = CurrentInstructionSlotIdx + std::max(1U, Current.NumSlots);
  • unsigned NextSlotIdx =
  •  CurrentInstructionSlotIdx + std::max(1U, Current.NumSlots);
    
    return NextSlotIdx % Queue.size();
    }

@@ -82,12 +83,13 @@ void RetireControlUnit::consumeCurrentToken() {
CurrentInstructionSlotIdx += std::max(1U, Current.NumSlots);
CurrentInstructionSlotIdx %= Queue.size();
AvailableEntries += Current.NumSlots;

  • Current = { InstRef(), 0U, false };
  • Current = {InstRef(), 0U, false};
    }

void RetireControlUnit::onInstructionExecuted(unsigned TokenID) {
assert(Queue.size() > TokenID);

  • assert(Queue[TokenID].IR.getInstruction() && "Instruction was not dispatched!");
  • assert(Queue[TokenID].IR.getInstruction() &&

  •     "Instruction was not dispatched!");
    

    assert(Queue[TokenID].Executed == false && "Instruction already executed!");
    Queue[TokenID].Executed = true;
    }
    diff --git a/llvm/lib/MCA/Stages/InstructionTables.cpp b/llvm/lib/MCA/Stages/InstructionTables.cpp
    index 937cc7da8de7249..270f08ecb33c746 100644
    --- a/llvm/lib/MCA/Stages/InstructionTables.cpp
    +++ b/llvm/lib/MCA/Stages/InstructionTables.cpp
    @@ -24,8 +24,7 @@ Error InstructionTables::execute(InstRef &IR) {
    UsedResources.clear();

    // Identify the resources consumed by this instruction.

  • for (const std::pair<uint64_t, ResourceUsage> &Resource :
  •   Desc.Resources) {
    
  • for (const std::pair<uint64_t, ResourceUsage> &Resource : Desc.Resources) {
    // Skip zero-cycle resources (i.e., unused resources).
    if (!Resource.second.size())
    continue;

@joker-eph
Copy link
Collaborator Author

@llvm/pr-subscribers-mlir-async

Changes

Test description...

Line 2

--
Full diff: https://github.com/llvm/llvm-project/pull/65439.diff

13 Files Affected:

  • (modified) llvm/docs/DeveloperPolicy.rst (+23-11)
  • (modified) llvm/lib/FileCheck/CMakeLists.txt (+4-6)
  • (modified) llvm/lib/FileCheck/FileCheck.cpp (+13-11)
  • (modified) llvm/lib/Frontend/CMakeLists.txt (+1-3)
  • (modified) llvm/lib/Frontend/HLSL/CMakeLists.txt (+7-11)
  • (modified) llvm/lib/Frontend/OpenACC/CMakeLists.txt (+7-10)
  • (modified) llvm/lib/Frontend/OpenMP/CMakeLists.txt (+8-20)
  • (modified) llvm/lib/FuzzMutate/CMakeLists.txt (+17-31)
  • (modified) llvm/lib/Fuzzer/README.txt (+1-1)
  • (modified) llvm/lib/InterfaceStub/CMakeLists.txt (+3-11)
  • (modified) llvm/lib/MCA/HardwareUnits/LSUnit.cpp (+5-7)
  • (modified) llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp (+5-3)
  • (modified) llvm/lib/MCA/Stages/InstructionTables.cpp (+1-2)
diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index a453601d7ad93d1..3d25ce9488f126e 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -46,21 +46,33 @@ quality.
 Stay Informed
 -------------
 
-Developers should stay informed by reading the `LLVM Discourse forums`_.
-If you are doing anything more than just casual work on LLVM, it is suggested that you also
-subscribe to the "commits" mailing list for the subproject you're interested in,
+Developers should stay informed by reading the `LLVM Discourse forums`_ and subscribing
+to the categories of interest for notifications.
+
+Paying attention to changes being made by others is a good way to see what other people
+are interested in and watching the flow of the project as a whole.
+
+Contibutions to the project are made through :ref:`GitHub Pull Requests `.
+You can subscribe to notification for areas of the codebase by joining
+one of the `pr-subscribers-* `_
+GitHub teams. This `mapping `_
+indicates which team is associated with a particular paths in the repository.
+
+You can also subscribe to the "commits" mailing list for a subproject you're interested in,
 such as `llvm-commits
 `_, `cfe-commits
 `_, or `lldb-commits
-`_.  Reading the
-"commits" list and paying attention to changes being made by others is a good
-way to see what other people are interested in and watching the flow of the
-project as a whole.
-
-We recommend that active developers monitor incoming issues to our `GitHub issue tracker `_ and preferably subscribe to the `llvm-bugs
+`_.
+
+Missing features and bugs are tracked through our `GitHub issue tracker `_
+and assigned labels. We recommend that active developers monitor incoming issues.
+You can subscribe for notification for specific components by joining
+one of the `issue-subscribers-* `_
+teams.
+You may also subscribe to the `llvm-bugs
 `_ email list to keep track
-of bugs and enhancements occurring in LLVM.  We really appreciate people who are
-proactive at catching incoming bugs in their components and dealing with them
+of bugs and enhancements occurring in the entire project.  We really appreciate people
+who are proactive at catching incoming bugs in their components and dealing with them
 promptly.
 
 Please be aware that all public LLVM mailing lists and discourse forums are public and archived, and
diff --git a/llvm/lib/FileCheck/CMakeLists.txt b/llvm/lib/FileCheck/CMakeLists.txt
index 91c80e1482f1985..bbc323b1f5c0449 100644
--- a/llvm/lib/FileCheck/CMakeLists.txt
+++ b/llvm/lib/FileCheck/CMakeLists.txt
@@ -1,8 +1,6 @@
-add_llvm_component_library(LLVMFileCheck
-  FileCheck.cpp
+add_llvm_component_library(LLVMFileCheck FileCheck.cpp
 
-  ADDITIONAL_HEADER_DIRS
-  "${LLVM_MAIN_INCLUDE_DIR}/llvm/FileCheck"
-)
+                               ADDITIONAL_HEADER_DIRS
+                           "${LLVM_MAIN_INCLUDE_DIR}/llvm/FileCheck")
 
-target_link_libraries(LLVMFileCheck LLVMSupport)
+    target_link_libraries(LLVMFileCheck LLVMSupport)
diff --git a/llvm/lib/FileCheck/FileCheck.cpp b/llvm/lib/FileCheck/FileCheck.cpp
index 49fda8fb63cd36c..129699169d84014 100644
--- a/llvm/lib/FileCheck/FileCheck.cpp
+++ b/llvm/lib/FileCheck/FileCheck.cpp
@@ -9,8 +9,8 @@
 // FileCheck does a line-by line check of a file that validates whether it
 // contains the expected content.  This is useful for regression tests etc.
 //
-// This file implements most of the API that will be used by the FileCheck utility
-// as well as various unittests.
+// This file implements most of the API that will be used by the FileCheck
+// utility as well as various unittests.
 //===----------------------------------------------------------------------===//
 
 #include "llvm/FileCheck/FileCheck.h"
@@ -307,7 +307,7 @@ Pattern::parseVariable(StringRef &Str, const SourceMgr &SM) {
 
   StringRef Name = Str.take_front(I);
   Str = Str.substr(I);
-  return VariableProperties {Name, IsPseudo};
+  return VariableProperties{Name, IsPseudo};
 }
 
 // StringRef holding all characters considered as horizontal whitespaces by
@@ -1847,20 +1847,22 @@ bool FileCheck::readCheckFile(
 
     // Verify that CHECK-LABEL lines do not define or use variables
     if ((CheckTy == Check::CheckLabel) && P.hasVariable()) {
-      SM.PrintMessage(
-          SMLoc::getFromPointer(UsedPrefixStart), SourceMgr::DK_Error,
-          "found '" + UsedPrefix + "-LABEL:'"
-                                   " with variable definition or use");
+      SM.PrintMessage(SMLoc::getFromPointer(UsedPrefixStart),
+                      SourceMgr::DK_Error,
+                      "found '" + UsedPrefix +
+                          "-LABEL:'"
+                          " with variable definition or use");
       return true;
     }
 
-    // Verify that CHECK-NEXT/SAME/EMPTY lines have at least one CHECK line before them.
+    // Verify that CHECK-NEXT/SAME/EMPTY lines have at least one CHECK line
+    // before them.
     if ((CheckTy == Check::CheckNext || CheckTy == Check::CheckSame ||
          CheckTy == Check::CheckEmpty) &&
         CheckStrings->empty()) {
-      StringRef Type = CheckTy == Check::CheckNext
-                           ? "NEXT"
-                           : CheckTy == Check::CheckEmpty ? "EMPTY" : "SAME";
+      StringRef Type = CheckTy == Check::CheckNext    ? "NEXT"
+                       : CheckTy == Check::CheckEmpty ? "EMPTY"
+                                                      : "SAME";
       SM.PrintMessage(SMLoc::getFromPointer(UsedPrefixStart),
                       SourceMgr::DK_Error,
                       "found '" + UsedPrefix + "-" + Type +
diff --git a/llvm/lib/Frontend/CMakeLists.txt b/llvm/lib/Frontend/CMakeLists.txt
index fa48c975a8b3e5a..231726c49590ee6 100644
--- a/llvm/lib/Frontend/CMakeLists.txt
+++ b/llvm/lib/Frontend/CMakeLists.txt
@@ -1,3 +1 @@
-add_subdirectory(HLSL)
-add_subdirectory(OpenACC)
-add_subdirectory(OpenMP)
+add_subdirectory(HLSL) add_subdirectory(OpenACC) add_subdirectory(OpenMP)
diff --git a/llvm/lib/Frontend/HLSL/CMakeLists.txt b/llvm/lib/Frontend/HLSL/CMakeLists.txt
index eda6cb8e69a4971..980d9118acdcf48 100644
--- a/llvm/lib/Frontend/HLSL/CMakeLists.txt
+++ b/llvm/lib/Frontend/HLSL/CMakeLists.txt
@@ -1,14 +1,10 @@
-add_llvm_component_library(LLVMFrontendHLSL
-  HLSLResource.cpp
+add_llvm_component_library(LLVMFrontendHLSL HLSLResource.cpp
 
-  ADDITIONAL_HEADER_DIRS
-  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend
-  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend/HLSL
+                               ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
+                           llvm / Frontend ${LLVM_MAIN_INCLUDE_DIR} / llvm /
+                           Frontend /
+                           HLSL
 
-  DEPENDS
-  intrinsics_gen
+                               DEPENDS intrinsics_gen
 
-  LINK_COMPONENTS
-  Core
-  Support
-  )
+                                   LINK_COMPONENTS Core Support)
diff --git a/llvm/lib/Frontend/OpenACC/CMakeLists.txt b/llvm/lib/Frontend/OpenACC/CMakeLists.txt
index f35201497869058..348d07347c79018 100644
--- a/llvm/lib/Frontend/OpenACC/CMakeLists.txt
+++ b/llvm/lib/Frontend/OpenACC/CMakeLists.txt
@@ -1,13 +1,10 @@
-add_llvm_component_library(LLVMFrontendOpenACC
-  ACC.cpp
+add_llvm_component_library(LLVMFrontendOpenACC ACC.cpp
 
-  ADDITIONAL_HEADER_DIRS
-  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend
-  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend/OpenACC
+                               ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
+                           llvm / Frontend ${LLVM_MAIN_INCLUDE_DIR} / llvm /
+                           Frontend /
+                           OpenACC
 
-  DEPENDS
-  acc_gen
-)
-
-target_link_libraries(LLVMFrontendOpenACC LLVMSupport)
+                               DEPENDS acc_gen)
 
+    target_link_libraries(LLVMFrontendOpenACC LLVMSupport)
diff --git a/llvm/lib/Frontend/OpenMP/CMakeLists.txt b/llvm/lib/Frontend/OpenMP/CMakeLists.txt
index a2eedabc3ed6903..35f7192c5fe9553 100644
--- a/llvm/lib/Frontend/OpenMP/CMakeLists.txt
+++ b/llvm/lib/Frontend/OpenMP/CMakeLists.txt
@@ -1,23 +1,11 @@
-add_llvm_component_library(LLVMFrontendOpenMP
-  OMP.cpp
-  OMPContext.cpp
-  OMPIRBuilder.cpp
+add_llvm_component_library(
+    LLVMFrontendOpenMP OMP.cpp OMPContext.cpp OMPIRBuilder.cpp
 
-  ADDITIONAL_HEADER_DIRS
-  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend
-  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Frontend/OpenMP
+        ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
+    llvm / Frontend ${LLVM_MAIN_INCLUDE_DIR} / llvm / Frontend /
+    OpenMP
 
-  DEPENDS
-  intrinsics_gen
-  omp_gen
+        DEPENDS intrinsics_gen omp_gen
 
-  LINK_COMPONENTS
-  Core
-  Support
-  TargetParser
-  TransformUtils
-  Analysis
-  MC
-  Scalar
-  BitReader
-  )
+            LINK_COMPONENTS Core Support TargetParser TransformUtils Analysis MC
+                Scalar BitReader)
diff --git a/llvm/lib/FuzzMutate/CMakeLists.txt b/llvm/lib/FuzzMutate/CMakeLists.txt
index e162c1bbe8e1078..4fa1aa2bed6eb96 100644
--- a/llvm/lib/FuzzMutate/CMakeLists.txt
+++ b/llvm/lib/FuzzMutate/CMakeLists.txt
@@ -1,37 +1,23 @@
-# Generic helper for fuzzer binaries.
-# This should not depend on LLVM IR etc.
-add_llvm_component_library(LLVMFuzzerCLI
-  FuzzerCLI.cpp
-  PARTIAL_SOURCES_INTENDED
+#Generic helper for fuzzer binaries.
+#This should not depend on LLVM IR etc.
+add_llvm_component_library(LLVMFuzzerCLI FuzzerCLI.cpp PARTIAL_SOURCES_INTENDED
 
-  ADDITIONAL_HEADER_DIRS
-  ${LLVM_MAIN_INCLUDE_DIR}/llvm/FuzzMutate
+                               ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
+                           llvm /
+                           FuzzMutate
 
-  LINK_COMPONENTS
-  Support
-  TargetParser
-  )
+                               LINK_COMPONENTS Support TargetParser)
 
-# Library for using LLVM IR together with fuzzers.
-add_llvm_component_library(LLVMFuzzMutate
-  IRMutator.cpp
-  OpDescriptor.cpp
-  Operations.cpp
-  RandomIRBuilder.cpp
+#Library for using LLVM IR together with fuzzers.
+    add_llvm_component_library(
+        LLVMFuzzMutate IRMutator.cpp OpDescriptor.cpp Operations
+            .cpp RandomIRBuilder.cpp
 
-  ADDITIONAL_HEADER_DIRS
-  ${LLVM_MAIN_INCLUDE_DIR}/llvm/FuzzMutate
+                ADDITIONAL_HEADER_DIRS ${LLVM_MAIN_INCLUDE_DIR} /
+        llvm /
+        FuzzMutate
 
-  DEPENDS
-  intrinsics_gen
+            DEPENDS intrinsics_gen
 
-  LINK_COMPONENTS
-  Analysis
-  BitReader
-  BitWriter
-  Core
-  Scalar
-  Support
-  TargetParser
-  TransformUtils
-  )
+                LINK_COMPONENTS Analysis BitReader BitWriter Core Scalar Support
+                    TargetParser TransformUtils)
diff --git a/llvm/lib/Fuzzer/README.txt b/llvm/lib/Fuzzer/README.txt
index 53ac637638f64ea..de140b0406de1f2 100644
--- a/llvm/lib/Fuzzer/README.txt
+++ b/llvm/lib/Fuzzer/README.txt
@@ -1 +1 @@
-libFuzzer was moved to compiler-rt in  https://reviews.llvm.org/D36908.
+libFuzzer was moved to compiler - rt in https: // reviews.llvm.org/D36908.
diff --git a/llvm/lib/InterfaceStub/CMakeLists.txt b/llvm/lib/InterfaceStub/CMakeLists.txt
index 68fe067701f23f4..c5997d1922f5a80 100644
--- a/llvm/lib/InterfaceStub/CMakeLists.txt
+++ b/llvm/lib/InterfaceStub/CMakeLists.txt
@@ -1,12 +1,4 @@
-add_llvm_component_library(LLVMInterfaceStub
-  ELFObjHandler.cpp
-  IFSHandler.cpp
-  IFSStub.cpp
+add_llvm_component_library(
+    LLVMInterfaceStub ELFObjHandler.cpp IFSHandler.cpp IFSStub.cpp
 
-  LINK_COMPONENTS
-  BinaryFormat
-  MC
-  Object
-  Support
-  TargetParser
-)
+        LINK_COMPONENTS BinaryFormat MC Object Support TargetParser)
diff --git a/llvm/lib/MCA/HardwareUnits/LSUnit.cpp b/llvm/lib/MCA/HardwareUnits/LSUnit.cpp
index bdc8b3d0e390e21..3b54cc2c8fddf73 100644
--- a/llvm/lib/MCA/HardwareUnits/LSUnit.cpp
+++ b/llvm/lib/MCA/HardwareUnits/LSUnit.cpp
@@ -96,8 +96,8 @@ unsigned LSUnit::dispatch(const InstRef &IR) {
     if (CurrentStoreBarrierGroupID) {
       MemoryGroup &StoreGroup = getGroup(CurrentStoreBarrierGroupID);
       LLVM_DEBUG(dbgs() << "[LSUnit]: GROUP DEP: ("
-                        << CurrentStoreBarrierGroupID
-                        << ") --> (" << NewGID << ")\n");
+                        << CurrentStoreBarrierGroupID << ") --> (" << NewGID
+                        << ")\n");
       StoreGroup.addSuccessor(&NewGroup, true);
     }
 
@@ -110,7 +110,6 @@ unsigned LSUnit::dispatch(const InstRef &IR) {
       StoreGroup.addSuccessor(&NewGroup, !assumeNoAlias());
     }
 
-
     CurrentStoreGroupID = NewGID;
     if (IsStoreBarrier)
       CurrentStoreBarrierGroupID = NewGID;
@@ -165,8 +164,7 @@ unsigned LSUnit::dispatch(const InstRef &IR) {
     if (IsLoadBarrier) {
       if (ImmediateLoadDominator) {
         MemoryGroup &LoadGroup = getGroup(ImmediateLoadDominator);
-        LLVM_DEBUG(dbgs() << "[LSUnit]: GROUP DEP: ("
-                          << ImmediateLoadDominator
+        LLVM_DEBUG(dbgs() << "[LSUnit]: GROUP DEP: (" << ImmediateLoadDominator
                           << ") --> (" << NewGID << ")\n");
         LoadGroup.addSuccessor(&NewGroup, true);
       }
@@ -175,8 +173,8 @@ unsigned LSUnit::dispatch(const InstRef &IR) {
       if (CurrentLoadBarrierGroupID) {
         MemoryGroup &LoadGroup = getGroup(CurrentLoadBarrierGroupID);
         LLVM_DEBUG(dbgs() << "[LSUnit]: GROUP DEP: ("
-                          << CurrentLoadBarrierGroupID
-                          << ") --> (" << NewGID << ")\n");
+                          << CurrentLoadBarrierGroupID << ") --> (" << NewGID
+                          << ")\n");
         LoadGroup.addSuccessor(&NewGroup, true);
       }
     }
diff --git a/llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp b/llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp
index 9297f0c4fd7bf1b..78ec5d06e8fedf9 100644
--- a/llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp
+++ b/llvm/lib/MCA/HardwareUnits/RetireControlUnit.cpp
@@ -66,7 +66,8 @@ const RetireControlUnit::RUToken &RetireControlUnit::getCurrentToken() const {
 
 unsigned RetireControlUnit::computeNextSlotIdx() const {
   const RetireControlUnit::RUToken &Current = getCurrentToken();
-  unsigned NextSlotIdx = CurrentInstructionSlotIdx + std::max(1U, Current.NumSlots);
+  unsigned NextSlotIdx =
+      CurrentInstructionSlotIdx + std::max(1U, Current.NumSlots);
   return NextSlotIdx % Queue.size();
 }
 
@@ -82,12 +83,13 @@ void RetireControlUnit::consumeCurrentToken() {
   CurrentInstructionSlotIdx += std::max(1U, Current.NumSlots);
   CurrentInstructionSlotIdx %= Queue.size();
   AvailableEntries += Current.NumSlots;
-  Current = { InstRef(), 0U, false };
+  Current = {InstRef(), 0U, false};
 }
 
 void RetireControlUnit::onInstructionExecuted(unsigned TokenID) {
   assert(Queue.size() > TokenID);
-  assert(Queue[TokenID].IR.getInstruction() && "Instruction was not dispatched!");
+  assert(Queue[TokenID].IR.getInstruction() &&
+         "Instruction was not dispatched!");
   assert(Queue[TokenID].Executed == false && "Instruction already executed!");
   Queue[TokenID].Executed = true;
 }
diff --git a/llvm/lib/MCA/Stages/InstructionTables.cpp b/llvm/lib/MCA/Stages/InstructionTables.cpp
index 937cc7da8de7249..270f08ecb33c746 100644
--- a/llvm/lib/MCA/Stages/InstructionTables.cpp
+++ b/llvm/lib/MCA/Stages/InstructionTables.cpp
@@ -24,8 +24,7 @@ Error InstructionTables::execute(InstRef &IR) {
   UsedResources.clear();
 
   // Identify the resources consumed by this instruction.
-  for (const std::pair &Resource :
-       Desc.Resources) {
+  for (const std::pair &Resource : Desc.Resources) {
     // Skip zero-cycle resources (i.e., unused resources).
     if (!Resource.second.size())
       continue;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant