This revision was automatically updated to reflect the committed changes.
Closed by commit rC347350: [OpenMP] Update CHECK-DAG usage in for_codegen.cpp
(authored by jdenny, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D54764?vs=174816&id=174836#toc
Repository:
rC Clang
ABataev added a comment.
In https://reviews.llvm.org/D54764#1304606, @jdenny wrote:
> In https://reviews.llvm.org/D54764#1304573, @ABataev wrote:
>
> > I'm fine with the patch,
>
>
> Thanks for the quick reviews. I'll push soon.
>
> > the original intention was to handle a possible situation whe
jdenny added a comment.
In https://reviews.llvm.org/D54764#1304573, @ABataev wrote:
> I'm fine with the patch,
Thanks for the quick reviews. I'll push soon.
> the original intention was to handle a possible situation where we may have
> several debug locations with the same line numbers.
We
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
I'm fine with the patch, the original intention was to handle a possible
situation where we may have several debug locations with the same line numbers.
Repository:
rC Clang
https://revi
jdenny created this revision.
jdenny added a reviewer: ABataev.
Herald added a subscriber: guansong.
This patch adjusts a test not to depend on deprecated FileCheck
behavior that permits overlapping matches within a block of CHECK-DAG
directives. Thus, this patch also removes uses of FileCheck's