[PATCH] Preserving 'nonnull' metadata in SimplifyCFG

Philip Reames listmail at philipreames.com
Wed Oct 22 09:12:30 PDT 2014


Hi hfinkel,

The actual change in this review are trivial (1 line), but I'd like a second opinion on their legality from someone familiar with the context.  I *believe* that this is legal, but a confirmation would help.  The semantics we're going for are: 'nonnull' is preserved when two instructions are merged iff both have it.  

Also, anyone know a reason we're not merging metadata when sinking instructions but are merging when hoisting?

http://reviews.llvm.org/D5910

Files:
  lib/Transforms/Utils/SimplifyCFG.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5910.15246.patch
Type: text/x-patch
Size: 710 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141022/b601fbd2/attachment.bin>


More information about the llvm-commits mailing list