[PATCH] D102588: [UpdateTestChecks] Default --x86_scrub_rip to False

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 20 18:24:53 PDT 2021


pengfei added a comment.

> Updated LCPI regex to include Darwin style LCPI_[0-9]+_[0-9]+ (no
> leading dot).

Do we have a test for it?



================
Comment at: llvm/test/CodeGen/X86/abi-isel.ll:1615
 ; LINUX-64-STATIC:       # %bb.0: # %entry
-; LINUX-64-STATIC-NEXT:    movl dsrc+{{.*}}(%rip), %eax
-; LINUX-64-STATIC-NEXT:    movl %eax, ddst+{{.*}}(%rip)
+; LINUX-64-STATIC-NEXT:    movl dsrc+64(%rip), %eax
+; LINUX-64-STATIC-NEXT:    movl %eax, ddst+64(%rip)
----------------
Not sure how many testing scenarios like this. But I think this is the origin of this knob. Can we change it to something like
`SCRUB_X86_RIP_RE = re.compile(r'[.\d]+\(%rip\)')` instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102588



More information about the llvm-commits mailing list