[PATCH] D66604: [GVN] AnalyzeLoadAvailability: Replace a load after lifetime.end with undef (PR20811)

Shreyansh Chouhan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 14 22:25:18 PDT 2019


BK1603 marked an inline comment as done.
BK1603 added a comment.

In D66604#1668406 <https://reviews.llvm.org/D66604#1668406>, @fhahn wrote:

> Thanks! Could you also add another test where we have a call to lifetime.end, followed by another defining store, followed by a load? This should not get optimized by this patch.


I have added a test case for it. :)
Also are the name for the test files fine? Or should I change them?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66604





More information about the llvm-commits mailing list