teemperor added inline comments.
================ Comment at: lldb/test/API/commands/frame/recognizer/main.m:1 -#import <Foundation/Foundation.h> +#import <objc/NSObject.h> ---------------- teemperor wrote: > I guess this could also be just `stdio.h` (FWIW, the printfs in tests can > also be removed as they just require including system headers and are > essentially never useful). Just to be clear: I'm just saying that removing `printfs` and everything else that references system headers is usually always a good idea but I'm not asking that you should remove them here too. The `stdio.h` is perfect for this patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99867/new/ https://reviews.llvm.org/D99867 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
