[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