Differential D65265

[Loop Utils] Extend the scope of addStringMetadataToLoop
ClosedPublic
Actions

Authored by skatkov on Jul 25 2019, 1:25 AM.

Tags

Subscribers

hiraditya, llvm-commits

Details

Reviewers

reames
• Ashutosh

Commits

rG7f8c809592ed: [Loop Utils] Extend the scope of addStringMetadataToLoop.
rL367087: [Loop Utils] Extend the scope of addStringMetadataToLoop.

Summary

To avoid duplicates in loop metadata, if the string to add is
already there, just update the value.

Diff Detail

Repository: rL LLVM

Event Timeline

skatkov created this revision.Jul 25 2019, 1:25 AM

Herald added a project: Restricted Project. · View Herald TranscriptJul 25 2019, 1:25 AM

Herald added a subscriber: hiraditya. · View Herald Transcript

skatkov added a parent revision: D65264: [Loop Utils] Move utilty addStringMetadataToLoop to LoopUtils.cpp. NFC..Jul 25 2019, 1:25 AM

skatkov added a child revision: D64972: [Loop Peeling] Do not close further unroll/peel if profile based peeling was not used.Jul 25 2019, 1:29 AM

LGTM

This revision is now accepted and ready to land.Jul 25 2019, 10:58 AM

Closed by commit rL367087: [Loop Utils] Extend the scope of addStringMetadataToLoop. (authored by skatkov, committed by ). · Explain WhyJul 26 2019, 12:05 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			llvm/trunk/include/llvm/Transforms/Utils/LoopUtils.h (2 lines)
M			llvm/trunk/lib/Transforms/Utils/LoopUtils.cpp (20 lines)

Diff 211884

llvm/trunk/include/llvm/Transforms/Utils/LoopUtils.h

Loading...

llvm/trunk/lib/Transforms/Utils/LoopUtils.cpp

Loading...