MaskRay added a subscriber: rsmith.
MaskRay added inline comments.

================
Comment at: clang/test/CodeGen/basic-block-sections.c:31
 // BB_LABELS: world:
-// BB_LABELS: a.BB.world:
-// BB_LABELS: aa.BB.world:
-// BB_LABELS: a.BB.another:
+// BB_LABELS: .Lfunc_begin0:
+// BB_LABELS: .LBB_END0_0:
----------------
Assembly tests are not common in clang/test but @rsmith LGTMed D68049 so I 
think it should be fine. Now this file tests more machine level stuff. Should 
the file switch to LLVM IR?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85408

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

Reply via email to