[PATCH] D63356: UpdateTestChecks: Consider .section as end of function for AMDGPU

Fri Jun 14 13:05:09 PDT 2019

arsenm created this revision.
arsenm added reviewers: RKSimon, kzhuravl, scott.linder.
Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, wdng.

Kernels seem to go directly to a section switch instead of emitting
.Lfunc_end. This fixes including all of the kernel metadata in the
check lines, which is undesirable most of the time.


https://reviews.llvm.org/D63356

Files:
  utils/UpdateTestChecks/asm.py


Index: utils/UpdateTestChecks/asm.py
===================================================================

--- utils/UpdateTestChecks/asm.py
+++ utils/UpdateTestChecks/asm.py
@@ -39,7 +39,7 @@
     r'^_?(?P<func>[^:]+):[ \t]*;+[ \t]*@(?P=func)\n[^:]*?'
     r'(?P<body>.*?)\n' # (body of the function)
     # This list is incomplete
-    r'.Lfunc_end[0-9]+:\n',
+    r'^\s*(\.Lfunc_end[0-9]+:\n|\.section)',
     flags=(re.M | re.S))
 
 ASM_FUNCTION_HEXAGON_RE = re.compile(


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63356.204833.patch
Type: text/x-patch
Size: 473 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190614/828708c3/attachment.bin>
