[PATCH] D12456: [IR] Add operand bundles to CallInst and InvokeInst.

Dmitry Babokin via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 05:24:21 PDT 2015


Sanjoy,

Your comment in include/llvm/IR/InstrTypes.h in line 1137 is undeniably
nice, but it causes a warning (or an error when building with "-Wall"):

“error: multi-line comment [-Werror=comment]”

Could you please fix it? For example you can use | instead of \, or add
some "shielding" (|, ///, or whatever you prefer) in the leftmost position
of the whole pseudo graphics (i.e. lines 1137-1149.

Thanks!

Dmitry.

On Fri, Sep 25, 2015 at 2:57 AM, Sanjoy Das via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> sanjoy added a comment.
>
> Just to make it clear:  this was LGTM'ed by Duncan via email (for some
> reason these emails did not show up here).
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D12456
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151001/84fbae1f/attachment-0001.html>


More information about the llvm-commits mailing list