[PATCH] D153382: [lld-macho][test] Make reloc-subtractor.s robust

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 21 11:09:03 PDT 2023


MaskRay added inline comments.


================
Comment at: lld/test/MachO/reloc-subtractor.s:52
 
+.section __DATA,bar
+_minuend_1:
----------------
smeenai wrote:
> MaskRay wrote:
> > smeenai wrote:
> > > Is it important to repeat the `.section` directive?
> > Yes, the above check lines test this...
> > `# CHECK:       {{0*}}[[#%x, SUB1ADDR:]] l {{.*}} __DATA,bar _sub1`
> > 
> > 
> I meant that we already have a `.section __DATA,bar` above `_sub1`, so do we need to issue that directive again here?
Sorry, I missed this. I think this `.section __DATA,bar` directive can be removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153382



More information about the llvm-commits mailing list