[PATCH] D12464: Add way to test for builtin TargetOpcode

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 2 11:31:18 PDT 2015


arsenm added a comment.

In http://reviews.llvm.org/D12464#258300, @ab wrote:

> With '<' -> '<=', seems reasonable.
>
> Can you add a comment?  I'm not sure we refer to these as "builtins" anywhere else (not that I have a better name though).


I got this from the BUILTIN_OP_END. The comment at the start of the enum calls these "Invariant opcodes" but that doesn't sound good. How about isGenericOpcode()?


http://reviews.llvm.org/D12464





More information about the llvm-commits mailing list