DavidSpickett accepted this revision. DavidSpickett added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: llvm/test/DebugInfo/COFF/objc.ll:4-6 +; ASM: .short 4412 # Record kind: S_COMPILE3 +; ASM-NEXT: .long 17 # Flags and language +; ASM-NEXT: .short 208 # CPUType ---------------- sgraenitz wrote: > DavidSpickett wrote: > > DavidSpickett wrote: > > > sgraenitz wrote: > > > > DavidSpickett wrote: > > > > > Are these checked, should they be? > > > > Thanks for taking a look! What we check here is the `17` in language > > > > flags. The rest is just to make sure we get the right context. (Same > > > > for the others.) Is that your question? > > > Right but I don't know what that adds once you get beyond the Language > > > line. It's a small thing, so that the test doesn't fail if a new key is > > > added. > > Ignore the previous comment, wrong line. > > > > So yes you can check these, but no one actually does. If you look at the > > filecheck line it only does OBJ. > > > > Same for the rust change you linked. Unless there is some implicit > > behaviour here. One way to find out, put a bogus value in there and it > > should fail. > The first RUN line will exercise the ASM checks: > ``` > ; RUN: llc < %s | FileCheck %s --check-prefix=ASM > ``` Doh, sometimes I forget how to read. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146221/new/ https://reviews.llvm.org/D146221 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits