[PATCH] D62850: [X86] Fix builtins-x86.c test where it wasn't using immediates. NFC

Russell Gallop via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 4 07:03:25 PDT 2019


russell.gallop added a comment.

> Is the compiler missing a check that these parameters are immediates?

I don't think that there can be a check, or this would have been noticed.

I don't know whether this is possible and/or desirable. Do users use one version of the builtin and want the compiler to decide whether to use the immediate form?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62850/new/

https://reviews.llvm.org/D62850





More information about the cfe-commits mailing list