[PATCH] D148129: [GVN] Adjust metadata for coerced load CSE
luxufan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 16 01:03:06 PDT 2023
StephenFan accepted this revision.
StephenFan added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:1008
+ // unless the load has !noundef, in which case all metadata violations
+ // will be promoted to UB.
+ if (!CoercedLoad->hasMetadata(LLVMContext::MD_noundef))
----------------
For !noalias or !alias.scope, I think we don't care about the size and type. But it is also dropped. Maybe leave a TODO which says we can do better for these two metadata.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148129/new/
https://reviews.llvm.org/D148129
More information about the llvm-commits
mailing list