diff options
| author | Eric Liu <ioeric@google.com> | 2016-09-17 12:26:42 +0000 |
|---|---|---|
| committer | Eric Liu <ioeric@google.com> | 2016-09-17 12:26:42 +0000 |
| commit | e9906525423ac47e774be1d52432f8033ec89486 (patch) | |
| tree | 6c18423b1e4e24223c7fef87ad1564ee3b1a2d62 /llvm/test/Bitcode/metadata-strings.ll | |
| parent | dfbbbc86a1be7d3d2c22d89bfef7cb9914756c59 (diff) | |
| download | bcm5719-llvm-e9906525423ac47e774be1d52432f8033ec89486.tar.gz bcm5719-llvm-e9906525423ac47e774be1d52432f8033ec89486.zip | |
When replacements have the same offset, make replacements with smaller length order first in the set.
Summary:
No behavioral change intended. The change makes iterating the replacements set more intuitive in Replacements class implementation. Previously, insertion is ordered before an deletion/replacement with the same offset, which is counter-intuitive for implementation, especially for a followup patch to support adding insertions around replacements.
With the current ordering, we only need to make `applyAllReplacements` iterate the replacements set reversely when applying them so that deletion/replacement is still applied before insertion with the same offset.
Reviewers: klimek, djasper
Subscribers: klimek, cfe-commits
Differential Revision: https://reviews.llvm.org/D24663
llvm-svn: 281819
Diffstat (limited to 'llvm/test/Bitcode/metadata-strings.ll')
0 files changed, 0 insertions, 0 deletions
