barannikov88 added a comment.

In D144218#4255686 <https://reviews.llvm.org/D144218#4255686>, @aaron.ballman 
wrote:

> In D144218#4254021 <https://reviews.llvm.org/D144218#4254021>, @barannikov88 
> wrote:
>
>> There is something wrong with the `#if`s in the test, isn't it? They all 
>> look no-op.
>
> Those are verifying that the macro expands to a value that can be used by the 
> preprocessor to avoid issues like the one pointed out in this comment: 
> https://reviews.llvm.org/D144218#inline-1396599

Ah, I see, thanks. I should have read the comments more carefully.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144218

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to