Skip to content

Commit 697082d

Browse files
[Scalar] Use LLVMContext::MD_mem_parallel_loop_access directly (NFC) (#69549)
This patch "constant propagates" LLVMContext::MD_mem_parallel_loop_access into wherever ParallelLoopAccessMDKind is used.
1 parent a7c4ff9 commit 697082d

File tree

1 file changed

+7
-12
lines changed

1 file changed

+7
-12
lines changed

llvm/lib/Transforms/Scalar/Scalarizer.cpp

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -282,12 +282,10 @@ T getWithDefaultOverride(const cl::opt<T> &ClOption,
282282

283283
class ScalarizerVisitor : public InstVisitor<ScalarizerVisitor, bool> {
284284
public:
285-
ScalarizerVisitor(unsigned ParallelLoopAccessMDKind, DominatorTree *DT,
286-
ScalarizerPassOptions Options)
287-
: ParallelLoopAccessMDKind(ParallelLoopAccessMDKind), DT(DT),
288-
ScalarizeVariableInsertExtract(
289-
getWithDefaultOverride(ClScalarizeVariableInsertExtract,
290-
Options.ScalarizeVariableInsertExtract)),
285+
ScalarizerVisitor(DominatorTree *DT, ScalarizerPassOptions Options)
286+
: DT(DT), ScalarizeVariableInsertExtract(getWithDefaultOverride(
287+
ClScalarizeVariableInsertExtract,
288+
Options.ScalarizeVariableInsertExtract)),
291289
ScalarizeLoadStore(getWithDefaultOverride(ClScalarizeLoadStore,
292290
Options.ScalarizeLoadStore)),
293291
ScalarizeMinBits(getWithDefaultOverride(ClScalarizeMinBits,
@@ -337,8 +335,6 @@ class ScalarizerVisitor : public InstVisitor<ScalarizerVisitor, bool> {
337335

338336
SmallVector<WeakTrackingVH, 32> PotentiallyDeadInstrs;
339337

340-
unsigned ParallelLoopAccessMDKind;
341-
342338
DominatorTree *DT;
343339

344340
const bool ScalarizeVariableInsertExtract;
@@ -448,8 +444,7 @@ bool ScalarizerLegacyPass::runOnFunction(Function &F) {
448444
return false;
449445

450446
DominatorTree *DT = &getAnalysis<DominatorTreeWrapperPass>().getDomTree();
451-
ScalarizerVisitor Impl(LLVMContext::MD_mem_parallel_loop_access, DT,
452-
ScalarizerPassOptions());
447+
ScalarizerVisitor Impl(DT, ScalarizerPassOptions());
453448
return Impl.visit(F);
454449
}
455450

@@ -556,7 +551,7 @@ bool ScalarizerVisitor::canTransferMetadata(unsigned Tag) {
556551
|| Tag == LLVMContext::MD_invariant_load
557552
|| Tag == LLVMContext::MD_alias_scope
558553
|| Tag == LLVMContext::MD_noalias
559-
|| Tag == ParallelLoopAccessMDKind
554+
|| Tag == LLVMContext::MD_mem_parallel_loop_access
560555
|| Tag == LLVMContext::MD_access_group);
561556
}
562557

@@ -1253,7 +1248,7 @@ bool ScalarizerVisitor::finish() {
12531248

12541249
PreservedAnalyses ScalarizerPass::run(Function &F, FunctionAnalysisManager &AM) {
12551250
DominatorTree *DT = &AM.getResult<DominatorTreeAnalysis>(F);
1256-
ScalarizerVisitor Impl(LLVMContext::MD_mem_parallel_loop_access, DT, Options);
1251+
ScalarizerVisitor Impl(DT, Options);
12571252
bool Changed = Impl.visit(F);
12581253
PreservedAnalyses PA;
12591254
PA.preserve<DominatorTreeAnalysis>();

0 commit comments

Comments
 (0)