[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