[PATCH] D18546: Prevent X86IselLowering from merging volatile loads

Nirav Dave via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 29 06:07:25 PDT 2016


niravd created this revision.
niravd added a reviewer: RKSimon.
niravd added a subscriber: llvm-commits.

In EltsFromConsecutiveLoads we check to see if a vector of values coming
from a set of loads can be merged into a single large load, but did not
check that the loads are non-volatile.

This issue is exposed by the Merge Consecutive Store cleanups in D14834

http://reviews.llvm.org/D18546

Files:
  lib/Target/X86/X86ISelLowering.cpp
  test/CodeGen/X86/merge-consecutive-loads-128.ll
  test/CodeGen/X86/merge-consecutive-loads-256.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18546.51894.patch
Type: text/x-patch
Size: 4903 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160329/8624f7b2/attachment.bin>


More information about the llvm-commits mailing list