[PATCH] D71241: [OpenMP][WIP] Use overload centric declare variants

[Johannes Doerfert via Phabricator via cfe-commits]

jdoerfert added a comment.

In D71241#1787888 <https://reviews.llvm.org/D71241#1787888>, @ABataev wrote:

> Hal, are we going to support something like this?


I'm not Hal but I will answer anyway.

>   void cpu() { asm("nop"); }
>   
>   #pragma omp declare variant(cpu) match(device = {kind(cpu)})
>   void wrong_asm() {
>     asm ("xxx");
>   }
> 
> 
> Currently, there is an error when we try to emit the assembler output.

While it is unclear to me what you think should happen, an error pointing at `"xxx"` is to be expected without further information on how this is compiled.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71241