[llvm-bugs] [Bug 40117] New: Copying of new LLVMContext::MD_access_group metadata missing in combineStoreToNewValue()

[via llvm-bugs]

https://bugs.llvm.org/show_bug.cgi?id=40117

            Bug ID: 40117
           Summary: Copying of new LLVMContext::MD_access_group metadata
                    missing in combineStoreToNewValue()
           Product: libraries
           Version: trunk
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: Transformation Utilities
          Assignee: unassignedbugs at nondot.org
          Reporter: moritz.kreutzer at siemens.com
                CC: llvm-bugs at lists.llvm.org

Hi,

I had a look at the recently merged commit for adding the MD_access_group
metadata globally: https://reviews.llvm.org/rL349725

In InstCombine's combineLoadToNewType() method, the new metadata gets copied to
the new load instruction alongside of the old MD_mem_parallel_loop_access
metadata. However, in the seemingly very similar combineStoreToNewValue()
method (which I assume does the same thing for stores instead of loads), the
new metadata is not copied to the new store instruction, but it seems to me as
if it should (see InstCombineLoadStoreAlloca.cpp:555). While I'm not 100% sure
that this is a bug, it seems slightly suspicious to me. I'd be happy to be
proven wrong and this is a WONTFIX, though.


Moritz

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20181220/235feaa9/attachment-0001.html>