[SimplifyCFG] use helper for combining metadata

Timothy Arceri via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 19 22:02:35 PST 2017


Rather than redefining known ids everywhere use the existing helper.

The helper should probably be renamed but its already used elsewhere
in SimplifyCFG so I've left as is for now.


Separate from this patch (but related). I'm also after suggestions on 
how to handle this bug:
https://bugs.llvm.org//show_bug.cgi?id=32001

The problem is the combine just throws away any metadata it doesn't know 
about. I could probably skip the sink if there are unknow metadata ids 
for the instruction, but that won't stop the issue popping up elsewhere.

How would this kind of thing be handled normally in LLVM? Is there a way 
to hook in target cpu specific functions into the combine calls? If 
someone can point me to an example of how this is normally done in LLVM 
that would be great.

Thanks,
Tim
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm-use-helper.diff
Type: text/x-patch
Size: 1140 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170220/0d66c448/attachment.bin>


More information about the llvm-commits mailing list