jyu2 marked an inline comment as done.
jyu2 added a comment.
Thanks. @rnk
================
Comment at: clang/test/CodeGenCXX/ms-inheriting-ctor.cpp:41
+
+// CHECK-LABEL: define internal noundef ptr
@"??0?$B@_N@@QEAA@AEBVF@@AEBUA@@@Z"(ptr noundef nonnull returned align 1
dereferenceable(1) %this, ptr noundef nonnull align 1 dereferenceable(1) %0,
ptr noundef nonnull align 1 dereferenceable(1) %1) unnamed_addr #2 align 2
+// CHECK-LABEL: define linkonce_odr dso_local noundef ptr
@"??0?$c@_NUb@@@@QEAA@AEBVF@@AEBUA@@@Z"(ptr noundef nonnull returned align 1
dereferenceable(1) %this, ptr noundef nonnull align 1 dereferenceable(1) %p1,
ptr noundef nonnull align 1 dereferenceable(1) %d) unnamed_addr #2 comdat align
2
----------------
rnk wrote:
> jyu2 wrote:
> > rnk wrote:
> > > To make this less fragile, can you come up with a way to use `CHECK-NOT:
> > > comdat` since that's the key thing we're testing for here? You will need
> > > some subsequent anchor like `entry:` or something else.
> > Thanks. I changed.
> This isn't quite what I was trying to suggest, I was thinking something more
> like:
>
> ```
> // CHECK-LABEL: define internal noundef ptr
> @"??0?$B@_N@@QEAA@AEBVF@@AEBUA@@@Z"
> // CHECK-NOT: comdat
> // CHECK-SAME: {{\{$}}
> ```
>
> So, it finds the class with the inheriting constructor, and then checks that
> the word "comdat" does not appear anywhere on that line. I haven't tested
> the CHECK-SAME pattern, it may need some adjustment.
I see. Thanks!! I changed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158538/new/
https://reviews.llvm.org/D158538
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
