[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