[PATCH] D122205: [LoopIdiom] Merge TBAA of adjacent stores when creating memset

Stephen Long via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 22 06:41:17 PDT 2022


steplong added a comment.

In D122205#3399389 <https://reviews.llvm.org/D122205#3399389>, @jeroen.dobbelaere wrote:

> Unfortunately, the context of this patch is not available :(

Whoops, should have explained the context. We were seeing GVN remove a load that had a TBAA that matched the 2nd store because GVN determined that it didn't match the TBAA on the memset that was created. The memset had the TBAA of the first store.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122205/new/

https://reviews.llvm.org/D122205



More information about the llvm-commits mailing list