[PATCH] D127202: [InlineFunction] don't add noalias metadata for unknown objects

Jeroen Dobbelaere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 30 23:48:40 PDT 2022


jeroen.dobbelaere added inline comments.


================
Comment at: llvm/test/Transforms/Inline/inline-noalias-unidentify-object.ll:2
 ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
 ; RUN: opt < %s -passes=inline -enable-noalias-to-md-conversion -S | FileCheck %s
 
----------------
shchenz wrote:
> jeroen.dobbelaere wrote:
> > You'll need to add the option to check full lines, otherwise the check on line 17 (store i32...) will still match a store containing a noalias
> 92a3e1b5c9e7ddd1b981393eac391a3745eff9a8 is committed to address this comment.
> 
> Sorry, I didn't noticed this comment when I committed the patch, seems we are almost at the same time : )
Thx !


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127202



More information about the llvm-commits mailing list