closed by commit rC321474 (rL321474)

turning aes off forbids enabling vaes

turning off pclmul forbids enabling vpclmulqdq

tests

addressed comments

add forgotten undef[s]

removing guards to allow better diags

addressed Craig's comments

adopted changes proposed by Craig:
maskz variants are out in favor of explicit passing of the additive identity
BWI dependency is explicitly stated
GCC builtins counterparts are matched
+ nice formatting of feature recognition

adopted a stricter type constraints on bits shuffle node, as suggested by Craig

intrinsics support (3/3)

added tzcnt tests
removed redundant blank line

further refined selection of conflicting insns, as suggested by Craig (thanks)

Reid,
can you please have a look?

further refined selection of conflicting insns, as suggested by Craig
added commutation tests, as suggested by Simon

Separated VBMI2's concatenate/shift insns implementation course from PMADD52

refined VPCLMULQDQ recognition in favor of PCLMULQDQ

addressed Craig's comments

intel syntax encoding test is to be added shortly

In D37461#913703, @RKSimon wrote:

@coby What is happening with this patch?

thanks for the prompt review.
will address enum's values definition ordering upon the commit itself

Minimal source changes - introduced 'new' enum values definitions as adapted on dependent change (D37412)
Added few more assembly level testing, as requested by @rnk
Minimized 'offset_operator.ll' test-case, and auto-updated it checks